Skip to content

Use cross-rs#332

Merged
devigned merged 1 commit intocontainerd:mainfrom
jprendes:use-cross
Oct 3, 2023
Merged

Use cross-rs#332
devigned merged 1 commit intocontainerd:mainfrom
jprendes:use-cross

Conversation

@jprendes
Copy link
Copy Markdown
Collaborator

@jprendes jprendes commented Sep 22, 2023

This PR enables the use of cross-rs for building and testing.

make build-wasmedge CARGO=cross TARGET=x86_64-unknown-linux-musl

It also works as an example on how to set up the cross containers for building runwasi with libseccomp

This PR depends on #331 as otherwise wasmedge wouldn't build on musl targets.

Note that the release workflows are not using cross-rs which can be done in a follow up PR.

@jprendes jprendes force-pushed the use-cross branch 2 times, most recently from 64c75ee to 5f0947f Compare September 24, 2023 21:10
@jprendes
Copy link
Copy Markdown
Collaborator Author

We've used this to build the x86_64 and aarch64 binaries we ship with Docker Desktop.
https://github.com/rumpl/containerd-wasi-shims/blob/main/.github/workflows/build-runwasi.yml#L31

Signed-off-by: Jorge Prendes <jorge.prendes@gmail.com>
@jsturtevant
Copy link
Copy Markdown
Contributor

LGTM

Copy link
Copy Markdown
Contributor

@devigned devigned left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@devigned devigned merged commit ec65b3f into containerd:main Oct 3, 2023
@jprendes jprendes deleted the use-cross branch February 27, 2025 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants