Skip to content

Bump wasmtime to version 32#457

Merged
saulecabrera merged 2 commits into
mainfrom
wasmtime-32
Jun 3, 2025
Merged

Bump wasmtime to version 32#457
saulecabrera merged 2 commits into
mainfrom
wasmtime-32

Conversation

@saulecabrera

Copy link
Copy Markdown
Member

No description provided.

Comment thread ext/Cargo.toml
wasmtime-environ = "=31.0.0"
deterministic-wasi-ctx = "=0.1.34"
wasmtime-environ = "=32.0.0"
deterministic-wasi-ctx = { version = "=1.2.0", features = ["wasi-common"] }

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Until #448 is merged.

Comment thread spec/unit/trap_spec.rb
expect(trap.wasm_backtrace_message).to eq(<<~MSG.rstrip)
error while executing at wasm backtrace:
0: 0x1a - <unknown>!<wasm function 0>
0: 0x1a - <unknown>!<wasm function 0>

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The formatting here changed a bit, I wasn't able to pinpoint if this is intentional in upstream wasmtime though.

@saulecabrera

Copy link
Copy Markdown
Member Author

Ugh, I think the failures are related to linking.

@jeffcharles

Copy link
Copy Markdown
Collaborator

Changes so far look good to me. But interested in seeing what the fix for not being able to find stdio.h is.

@saulecabrera

Copy link
Copy Markdown
Member Author

The issues in rb-sys were fixed upstream (thanks Jeff!), so the Windows issues are now fixed. There's one Valgrind issue that is currently under investigation, so I'm proposing that we merge on red, and follow-up with a fix after the fact.

New versions of `rustix` don't use libc by default.
bytecodealliance/rustix#1389
@saulecabrera saulecabrera merged commit 0ccad30 into main Jun 3, 2025
13 checks passed
@saulecabrera

Copy link
Copy Markdown
Member Author

Memcheck issue was fixed in 20fb455

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.

2 participants