Skip to content

Conversation

dicej
Copy link
Collaborator

@dicej dicej commented Oct 28, 2024

An earlier commit which updated various dependencies (most notably, Wasmtime) unmasked a bug in add_wasi_and_stubs such that we were overwriting the host functions added by wasmtime-wasi with trapping stubs. This wasn't a problem previously because there was only one version of WASIp2: 0.2.0. Now that we have 0.2.1, 0.2.2, etc., with corresponding support in Wasmtime, we need to be a bit more careful.

An earlier commit which updated various dependencies (most notably, Wasmtime)
unmasked a bug in `add_wasi_and_stubs` such that we were overwriting the host
functions added by `wasmtime-wasi` with trapping stubs.  This wasn't a problem
previously because there was only one version of WASIp2: 0.2.0.  Now that we
have 0.2.1, 0.2.2, etc., with corresponding support in Wasmtime, we need to be a
bit more careful.

Signed-off-by: Joel Dice <joel.dice@fermyon.com>
@dicej dicej mentioned this pull request Oct 28, 2024
@dicej dicej merged commit 7a28de9 into main Oct 28, 2024
3 checks passed
@dicej dicej deleted the stub-fix branch October 28, 2024 17:50
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.

1 participant