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

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-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.

5 participants