Skip to content

Commit

Permalink
Merge pull request #486 from CosmWasm/alpine-builder-x86_64-filename
Browse files Browse the repository at this point in the history
Let upcoming alpine builder use an x86_64 specific file name
  • Loading branch information
webmaster128 authored Dec 1, 2023
2 parents eb3a5d4 + 8cf7e42 commit 582d9e6
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 1 deletion.
1 change: 1 addition & 0 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -384,6 +384,7 @@ jobs:
mkdir artifacts
# Static (from build)
# TODO: Change this line to libwasmvm_muslc.x86_64.a when upgrading to builders 0018+
cp ./internal/api/libwasmvm_muslc.a artifacts/libwasmvm_muslc.x86_64.a
cp ./internal/api/libwasmvm_muslc.aarch64.a artifacts/libwasmvm_muslc.aarch64.a
cp ./internal/api/libwasmvmstatic_darwin.a artifacts/
Expand Down
1 change: 1 addition & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,7 @@ release-build-alpine:
rm -rf libwasmvm/target/x86_64-unknown-linux-musl/release
# build the muslc *.a file
docker run --rm -u $(USER_ID):$(USER_GROUP) -v $(shell pwd)/libwasmvm:/code $(BUILDERS_PREFIX)-alpine
# TODO: Change this line to libwasmvm_muslc.x86_64.a when upgrading to builders 0018+
cp libwasmvm/artifacts/libwasmvm_muslc.a internal/api
cp libwasmvm/artifacts/libwasmvm_muslc.aarch64.a internal/api
make update-bindings
Expand Down
2 changes: 2 additions & 0 deletions builders/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,8 @@ versions of the builder images.
**Unreleased**

- Remove Go dev environment from `cosmwasm/go-ext-builder:XXXX-alpine`
- Write x86_64 muslc output in `libwasmvm_muslc.x86_64.a` instead of
`libwasmvm_muslc.a`

**Version 0017:**

Expand Down
2 changes: 1 addition & 1 deletion builders/guest/build_muslc.sh
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,4 @@ echo "Starting x86_64-unknown-linux-musl build"
cargo build --release --target x86_64-unknown-linux-musl --example wasmvmstatic

cp target/aarch64-unknown-linux-musl/release/examples/libwasmvmstatic.a artifacts/libwasmvm_muslc.aarch64.a
cp target/x86_64-unknown-linux-musl/release/examples/libwasmvmstatic.a artifacts/libwasmvm_muslc.a
cp target/x86_64-unknown-linux-musl/release/examples/libwasmvmstatic.a artifacts/libwasmvm_muslc.x86_64.a
2 changes: 2 additions & 0 deletions internal/api/link_muslc.go
Original file line number Diff line number Diff line change
Expand Up @@ -2,5 +2,7 @@

package api

// TODO: once we switch to builders 0018+, split this linking statement in x86_64 and arm64 like we do with the glibc case

// #cgo LDFLAGS: -Wl,-rpath,${SRCDIR} -L${SRCDIR} -lwasmvm_muslc
import "C"

0 comments on commit 582d9e6

Please sign in to comment.