-
Notifications
You must be signed in to change notification settings - Fork 55
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
Using Node.js 20, rake npm:ruby-head-wasm-wasi:check does not work #314
Comments
From my bisect, it looks like there should be something wrong in Node.js v19.3.0...v19.4.0. Here is the detailed backtrace for the segmentation fault.
|
I can confirm this doesn't work with Node.js 21 either. By the way, the fix to https://github.com/ruby/ruby.wasm/blob/main/packages/npm-packages/ruby-wasm-wasi/tools/run-test-unit.mjs#L30 (adding |
@terrablue Yes, that would be helpful :) |
@kateinoigakukun is it possible to run at least part of the tests or isolate this problem down to a particular test? I would like to start contributing, but it's hard to know if I'm introducing regressions otherwise. |
I think the easiest way at this time is just using Node.js older than 19. Or you can find how to execute a part of test suites by checking |
we are using node 20 LTS in production. |
I suspect nodejs/node#43697 includes something wrong |
What happened.
I installed Node.js 20.8.0 and ran
rake npm:ruby-head-wasm-wasi:check
and got the following error:It worked fine using Node.js 18.8.2.
What I tried
Added
version
property to the argument of the WASI constructor.ruby.wasm/packages/npm-packages/ruby-wasm-wasi/tools/run-test-unit.mjs
Lines 30 to 39 in 5a28621
I have tried both
unstable
andpreview1
according to https://nodejs.org/docs/latest-v20.x/api/wasi.html.The test ended with a
Segmentation fault
.The text was updated successfully, but these errors were encountered: