-
Notifications
You must be signed in to change notification settings - Fork 68
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
Check failed: result.second on Node.js v14.15.0 #47
Comments
I found a similar issue from nodejs, please see nodejs/node#32463 @addaleax would you please help to have a look? Thanks! |
@minggangw Without a smaller reproduction, I don’t think it’s going to be feasible for me to look into this in much detail |
Thanks for your reply and I will try to narrow it down to find any useful clue. |
@addalex Here's a script that breaks on macOS 11.0.1, using node 14.15.1 or 15.3.0. Works fine on node 12. #!/usr/bin/env node
const ffi = require('ffi-napi');
const ref = require('ref-napi');
const ArrayType = require('ref-array-napi');
const lib = ffi.Library(null, {
execvp: ['int', ['string', ArrayType('string')]],
dup2: ['int', ['int', 'int']],
});
[process.stdin, process.stdout, process.stderr].forEach((x, i) => {
lib.dup2(x._handle.fd, i);
});
lib.execvp('echo', ['', 'foo', ref.NULL]); Trace:
|
Folks, we too have run into this issue, any update on when this will be fixed pls? |
Same here. Works in 12, but fails in 14 and 15. |
Any update after more than a whole year? |
When I upgrade Node.js to the latest LTS v14.15.0, a fatal error happens when running the unit test, but it's ok with Node.js v12, please check out the error log below:
For the detailed log, please see https://travis-ci.org/github/RobotWebTools/rclnodejs/builds/739832615
Thanks!
The text was updated successfully, but these errors were encountered: