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

pkg/wamr : updated wamr version to v2.1.1 #20795

Merged
merged 1 commit into from
Jul 24, 2024
Merged

Conversation

bmewen
Copy link
Contributor

@bmewen bmewen commented Jul 24, 2024

A while ago I added the glue in WAMR to make RIOT works with its most recent version. Since the code is now implemented in the latest release I thought it could be good to update the version RIOT was using. I tested the example and had no particular problem on a DWM1001.

From what I saw the wamr package is for now only used in the example so that should not cause any extra problem. Feel free to test the example on any other board but be aware that if you are using your own program it may have been broken because of the slight API changes that occurred in wamr between v1.3.3 and v2.1.1 (calling a wasm function does not require the signature anymore if you are not using the iwasm API).

Contribution description

The only thing that was edited is the pkg/wamr/Makefile with the commit hash.

Issues/PRs references

Ref : wamr/PR/3508

@github-actions github-actions bot added the Area: pkg Area: External package ports label Jul 24, 2024
@mguetschow mguetschow added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Jul 24, 2024
@riot-ci
Copy link

riot-ci commented Jul 24, 2024

Murdock results

✔️ PASSED

36bcdd4 pkg/wamr : updated wamr version to v2.1.1

Success Failures Total Runtime
10196 0 10196 15m:33s

Artifacts

@kfessel
Copy link
Contributor

kfessel commented Jul 24, 2024

ran the example -> looks good
had a short look through the riotplatform changes in wamr -> looks fine

@benpicco benpicco added this pull request to the merge queue Jul 24, 2024
Merged via the queue into RIOT-OS:master with commit 16f8160 Jul 24, 2024
27 checks passed
@benpicco benpicco added this to the Release 2024.10 milestone Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: pkg Area: External package ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants