Stream: git-wasmtime

Topic: wasmtime / PR #7365 Enable the new preview1 implementatio...


view this post on Zulip Wasmtime GitHub notifications bot (Oct 25 2023 at 18:33):

alexcrichton requested pchickey for a review on PR #7365.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 25 2023 at 18:33):

alexcrichton opened PR #7365 from alexcrichton:preview2-by-default to bytecodealliance:main:

Currently Wasmtime has two implementations of the wasi_snapshot_preview1 set of APIs. The now-historical implementation lives in the wasi-common crate and the more recent implementation lives in the wasmtime-wasi crate. The main difference is that the wasmtime-wasi implementation is based on the implementation of preview2, meaning that the preview1 implementation is a shim in that direction. Additionally currently the preview2 implementation of preview1 is accessible via the -Spreview2 flag on the CLI.

This commit updates the interpretation of the -Spreview2 flag and the defaults around which implementation to choose. By default the preview1-built-on-preview2 implementation (the new wasmtime-wasi implementation) is selected now. This means that the wasi-common implementation is disabled by default. There are still two use cases to keep running the wasi-common implementation, however:

Tests using wasi_unstable now pass -Spreview2=n to explicitly request the old wasi-common implementation. Additionally the wasi-common implementation is still selected by default when -Sthreads is passed (enabling the WASI threads proposal).

<!--
Please make sure you include the following information:

Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.html

Please ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->

view this post on Zulip Wasmtime GitHub notifications bot (Oct 25 2023 at 18:33):

alexcrichton has marked PR #7365 as ready for review.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 25 2023 at 18:33):

alexcrichton requested wasmtime-core-reviewers for a review on PR #7365.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 27 2023 at 22:34):

pchickey submitted PR review:

Thanks. This is an exciting move! If we survive a release of this, I'll start moving the old implementation to be exported by an off-by-default wasmtime_wasi::legacy mod, and move ::preview2 to the root.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 27 2023 at 23:26):

pchickey merged PR #7365.


Last updated: Dec 23 2024 at 12:05 UTC