-
Notifications
You must be signed in to change notification settings - Fork 114
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The build is currently failing, https://travis-ci.org/MadLittleMods/node-usb-detection/builds/526592841
../src/detection.cpp:122:63: error: no member named 'ToChecked' in 'v8::Maybe<double>'
vid = (int) args[0]->NumberValue(Nan::GetCurrentContext()).ToChecked();
Do we need to update the nan
dependency?
We should also add a Node.js 12 build to Travis and Appveyor,
node-usb-detection/.travis.yml
Line 21 in d71db25
- BINARY_BUILDER="true" TRAVIS_NODE_VERSION="8" ARCH="x86" node-usb-detection/appveyor.yml
Line 10 in d71db25
nodejs_version: "8"
Sorry for spamming commits here, it's a lot of trial and error 😅 The only replacement I found for I tried adding Node 10 and 12 to Travis and AppVeyor, and ran into a few issues there:
|
FWIW, the ToChecked error will be fixed once nodejs/nan#852 is released. |
Thanks for the updates @kryops, all of the builds are green now ✅ Just to make sure and an extra set of eyes, have you sanity checked that the project works with Node.js v10 and Node.js v12? |
…older Node versions
Mhm, I'm all of a sudden getting a segfault in the Using it in my project works fine though on Node 10, 11 and 12. Test results: Node 10
Node 12
I run Archlinux with a 5.0 Kernel and udev through systemd 242.19 - no idea what's going on there. Here's the core dump of the segfault for completeness:
|
@kryops Are you running The relevant code for your segfault crash is here, node-usb-detection/src/detection_linux.cpp Line 229 in 46eaecc
Those tests were run on Ubuntu 16.04 in #46 I just ran the tests from I tried building with Node.js v12.2.0 but I am running into errors: Ubuntu 18.04, Node.js v12.2.0 `npm run rebuild`
Windows 10, Node.js v12.2.0 `npm run rebuild`
|
Interesting 🤔 Yes, I did a full I just tried to reproduce your build errors on my Windows 10 (Windows 10 Pro 1803, Version 10.0.17134 Build 17134) running Node v12.2.0, and both Are you sure you were on this branch and had reinstalled the dependencies to get the latest version of |
Thanks for this patch guys! FYI, this PR builds and runs without hassle on my config:
There are 25 warnings in src/detection.cpp indicating that functions 'Set' and 'Call' are now deprecated. |
Re-tested with deleting |
Thanks @kryops for the contribution ❤️! This will be part of Thanks @motla for the extra testing! |
Prebuilds not found for node 10, on windows: > usb-detection@4.3.0 install .\node_modules\usb-detection
> prebuild-install || node-gyp rebuild
prebuild-install WARN install No prebuilt binaries found (target=10.16.0 runtime=node arch=x64 libc= platform=win32) |
@jordao76 Can you create a new issue to track this? |
I am still facing the following issue: Error: The module '/opt/projects/workspace-edu-system/projects/electron-poc/node_modules/usb-detection/build/Release/detection.node' I am using, Please advise. |
Closes #80
There seem to be multiple ways to fix the NumberValue error, not sure which is the best/correct one.
I also did not check how this might affect compatibility with older versions of Node.