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

Update wasmi_wasi WASI implementing dependencies #943

Closed
Robbepop opened this issue Feb 28, 2024 · 3 comments · Fixed by #1140
Closed

Update wasmi_wasi WASI implementing dependencies #943

Robbepop opened this issue Feb 28, 2024 · 3 comments · Fixed by #1140
Labels
tech-debt An issue to resolve some technical debt.

Comments

@Robbepop
Copy link
Member

Robbepop commented Feb 28, 2024

Currently wasmi_wasi is stuck on v2.0 of wasi-common, wasi-cap-std-sync and wiggle crate.
For both wasi-common and wiggle this PR has shown that it is quite easy to update to the most recent version.

However, for wasi-cap-std-sync it is unfortunately not very easy to do so since this crate is hard wired with Wasmtime to work. However, for Wasmi we want to avoid having to depend or compile Wasmtime as dependency.

This issue is about resolving this problem so that we can use more recent WASI implementations with bug fixes and improvements.

This issue superseeds the following issue(s):

@Robbepop Robbepop added help wanted Extra attention is needed tech-debt An issue to resolve some technical debt. labels Feb 28, 2024
@Robbepop

This comment was marked as outdated.

@Robbepop
Copy link
Member Author

Robbepop commented Jul 2, 2024

This could be easily resolved with bytecodealliance/wasmtime#8894.

@Robbepop
Copy link
Member Author

Wasmtime's https://crates.io/crates/wasi-common/23.0.1 has just been released with the changes to its sync crate feature which should allow Wasmi to use it to replace the old wasi-common v2.0 and wasi-cap-std-sync v2.0 WASI dependencies with just wasi-common v23.0.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tech-debt An issue to resolve some technical debt.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant