-
-
Notifications
You must be signed in to change notification settings - Fork 30.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support a pthread and dynamic link build for wasm #96254
Comments
Thanks for your interest. 👍 The build rule is omitted on purpose. Last time I checked builds with dynamic linking and pthread proxying were unstable and crashed. Did you check that the target is passing all tests? |
There are some tests that fail (5 out of 435) but I thought this is all experimental anyway? Here's the failing tests:
I'm using this build rule to try and get https://github.com/Microsoft/DEBUGPY working in some form of a WASM build. Debugpy requires dynamic linking and pthreads. |
Would it be possible to add it and put a comment like this one? cpython/Tools/wasm/wasm_build.py Line 492 in caa2a97
|
That comment is outdated already :). wasm64-emscripten is working and all tests are passing -- or were passing Emscripten broke two days ago. Sam is working on a fix, emscripten-core/emscripten#17720
I would be willing to accept a |
For the record, pthreads and dynamic linking are experimental and currently broken for CPython. Every attempt to load a shared extension module leads to an immediate crash:
|
Closing as a duplicate of #101538 . |
Feature or enhancement
The tools/wasm/wasm_build.py is missing a build feature that supports pthreading and dynamic linking for node.
Pitch
Add this build feature
Previous discussion
The text was updated successfully, but these errors were encountered: