Skip to content

[STABLE] library: wasm32-wasip1-threads has functional pthreads#153634

Open
jieyouxu wants to merge 1 commit intorust-lang:stablefrom
jieyouxu:wasm32-wasip1-threads
Open

[STABLE] library: wasm32-wasip1-threads has functional pthreads#153634
jieyouxu wants to merge 1 commit intorust-lang:stablefrom
jieyouxu:wasm32-wasip1-threads

Conversation

@jieyouxu
Copy link
Member

@jieyouxu jieyouxu commented Mar 10, 2026

Minimal backport-suitable fix to avoid returning UNSUPPORTED_PLATFORM for wasm32-wasip1-threads which has a working pthreads implementation via emnapi.

The stable regression was reported in #153475.

This PR should be stable-backported-only (if accepted by T-libs) to mitigate #153475 on stable with a stable point release.

Minimal backport-suitable fix to avoid returning `UNSUPPORTED_PLATFORM`
for `wasm32-wasip1-threads` which has a working pthreads implementation
via `emnapi`.

The stable regression was reported in
<rust-lang#153475>.
@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Mar 10, 2026
@jieyouxu
Copy link
Member Author

@alexcrichton is this what you had in mind in #153475 (comment)?

@rust-log-analyzer
Copy link
Collaborator

A job failed! Check out the build log: (web) (plain enhanced) (plain)

Click to see the possible cause of the failure (guessed by this bot)

@alexcrichton
Copy link
Member

Indeed! I believe this would fix the issue while being less risky than backporting #152183

@jieyouxu jieyouxu marked this pull request as ready for review March 11, 2026 15:14
@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Mar 11, 2026
@rustbot
Copy link
Collaborator

rustbot commented Mar 11, 2026

⚠️ Warning ⚠️

  • Pull requests are usually filed against the main branch for this repo, but this one is against stable. Please double check that you specified the right target!

@jieyouxu
Copy link
Member Author

@rustbot reroll

Nominating for stable-backport only for mitigating #153475; the change here will need to be cherry-picked in a stable backport rollup w/ the patch version bump if this is accepted by libs.
@rustbot label: +stable-nominated

@rustbot rustbot added the stable-nominated Nominated for backporting to the compiler in the stable channel. label Mar 11, 2026
@the8472 the8472 added the stable-accepted Accepted for backporting to the compiler in the stable channel. label Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. stable-accepted Accepted for backporting to the compiler in the stable channel. stable-nominated Nominated for backporting to the compiler in the stable channel. T-libs Relevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants