Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support multi-arch / arm64 #693

Closed
wants to merge 3 commits into from
Closed

Conversation

mandrean
Copy link

This PR adds support for multi-arch / arm64 builds.

  • chore: Bump wasmvm to v0.16.5
  • build: Compile libwasmvm from source
  • ci: Build & publish multi-arch images

@kamsz
Copy link

kamsz commented Mar 16, 2022

hey @mandrean, i've been trying to test this on arm instances; however i noticed that LCD tend to hang after committing few blocks, is this something you perhaps experienced?

@mandrean
Copy link
Author

Hey @kamsz, yes it does indeed seem like wasmvm v0.16.5 and wasmer v2.2.0 introduced some breaking changes. So this should be seen as WIP until further notice 👍

@yun-yeo
Copy link
Contributor

yun-yeo commented Apr 1, 2022

@mandrean can you merge latest cosmwasm bump update?

@hanjukim
Copy link
Contributor

hanjukim commented Apr 7, 2022

This PR does not build the static linked of terrad with make build-linux. Maybe be due to cosmwasm update

@hanjukim
Copy link
Contributor

hanjukim commented Apr 7, 2022

This PR does not build the static linked of terrad with make build-linux

@hanjukim
Copy link
Contributor

hanjukim commented Apr 8, 2022

Static build was broken from wasmvm v0.16.6

1 similar comment
@hanjukim
Copy link
Contributor

hanjukim commented Apr 8, 2022

Static build was broken from wasmvm v0.16.6

@hanjukim hanjukim closed this Apr 8, 2022
@mandrean
Copy link
Author

@YunSuk-Yeo @hanjukim Sorry was away for ~2 weeks traveling.

This PR does not build the static linked of terrad with make build-linux. Maybe be due to cosmwasm update

Yes because of: CosmWasm/wasmvm#306

Might be fixed now, but using Alpine/Musl is questionable anyway. I would argue using Ubuntu or Debian + build-linux-with-shared-library is much preferred.

Size argument doesn't hold up any longer these days; Ubuntu Docker images are tiny (and usually cached on most machines anyway).

But let me get back to you! Might be fixed now.

@mandrean
Copy link
Author

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