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

Upgrade Wasmer to 2.2 (CosmWasm v1.0.0-beta6) #296

Merged
merged 4 commits into from Mar 7, 2022
Merged

Conversation

webmaster128
Copy link
Member

@webmaster128 webmaster128 commented Feb 23, 2022

This integrates the PR branch from CosmWasm/cosmwasm#1224.

Feel free to use this for testing. But be warned, heavy rebasing is going to happen here.

With the builders version 0009 the .dylib is a universal macOS library that works on both Intel and M1 Macs:

$ file api/libwasmvm.dylib 
api/libwasmvm.dylib: Mach-O universal binary with 2 architectures: [x86_64:Mach-O 64-bit dynamically linked shared library x86_64] [arm64]
api/libwasmvm.dylib (for architecture x86_64):  Mach-O 64-bit dynamically linked shared library x86_64
api/libwasmvm.dylib (for architecture arm64):   Mach-O 64-bit dynamically linked shared library arm64

When I check out this branch on a M1 machines, all make test tests pass except for the GetMetrics tests because 7.5% less memory is needed on ARM for whatever reason. All other tests pass.

@webmaster128 webmaster128 changed the title Test Wasmer 2.2 Test Wasmer 2.2 integration Feb 23, 2022
@webmaster128 webmaster128 mentioned this pull request Feb 23, 2022
@webmaster128 webmaster128 marked this pull request as ready for review March 7, 2022 16:24
@webmaster128 webmaster128 changed the title Test Wasmer 2.2 integration Upgrade Wasmer to 2.2 (CosmWasm v1.0.0-beta6) Mar 7, 2022
@webmaster128 webmaster128 merged commit ab31953 into main Mar 7, 2022
@webmaster128 webmaster128 deleted the wasmer-2.2 branch March 7, 2022 16:25
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.

None yet

1 participant