Stream: wit-bindgen

Topic: PR help please

view this post on Zulip Scott Waye (Nov 01 2025 at 20:39):

I have a problem with this PR https://github.com/bytecodealliance/wit-bindgen/pull/1346 . Lots of the test are failing, but the only changes are to the c# side, and locally

C:\github\wit-bindgen>cargo run test --languages rust,c tests/runtime     --artifacts target/artifacts     --rust-wit-bindgen-path ./crates/guest-rust
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.14s
     Running `target\debug\wit-bindgen.exe test --languages rust,c tests/runtime --artifacts target/artifacts --rust-wit-bindgen-path ./crates/guest-rust`
Building `wit-bindgen` from crates.io...
Testing if `c:\github\wasi-sdk27\bin/wasm32-wasip2-clang` works...
Compiling 133 components:
.....................................................................................................................................
Running 92 runtime tests:
............................................................................................
PASSED

The c tests pass, any idea what the failure in CI is? I see

         2: unknown handle index 1

But I don't know what it means...

[C#] Simple Async support by yowl · Pull Request #1346 · bytecodealliance/wit-bindgen
This PR add the very minimal changes to enable the first async functions tests to pass. These tests return immediately which means a lot of the callback, waitable, changes are not done yet. There...

view this post on Zulip Alex Crichton (Nov 01 2025 at 22:53):

looks like it's an issue unrelated to your changes, likely related to https://github.com/rust-lang/rust/issues/148347

Wasm Linker "module mismatch for symbol" Regression in 1.91.0 · Issue #148347 · rust-lang/rust
Additional notes and repro cases: https://github.com/posborne/rustc-1.91.0-wasm-linkage-bug-repro With the Rust 1.91.0 stable update, it appears that a regression has been introduced which causes l...

view this post on Zulip Alex Crichton (Nov 01 2025 at 22:53):

I pushed up a commit to try to pin rustc to an older version to resolve it

view this post on Zulip Alex Crichton (Nov 01 2025 at 23:25):

yay it merged!

Last updated: Dec 06 2025 at 07:03 UTC