-
Notifications
You must be signed in to change notification settings - Fork 40
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
Install Failed [ iNotify v1.4.4 | Node v13.5.0 | Node-Gyp v6.1.0 ] #75
Comments
I'm seeing the same with inotify-1.4.6 / node-12.16.1 / node-gyp 5.0.5. |
Same errors using inotify-1.4.6, node v14.2.0, npm 6.14.5, node-gyp 6.1.0. The pull request (repo: ashthespy/node-inotify) which should fix this failed with:
So I had to run In short, for Node v14 this should work: git clone https://github.com/ashthespy/node-inotify.git
cd node-inotify
npm install nan
node-gyp rebuild
cd ..
node -e "const Inotify = require('./node-inotify').Inotify;console.log(Inotify);" |
For anyone else who discovered this issue after googling their error message, for me this was fixed by switching to node v11.14.0 (previously I was using v14.8.0). At the time of writing, the inotify readme states that only nodejs versions up to v11.x.x are supported. |
@rileyweber13, thanks for the extra clarification! We added support for Node 14 in 5d12f64 but, unfortunately, I haven't had time to properly test it before tagging a release. |
I'm getting the same |
@vbraun Is this running of the current master? |
No, that was with the latest release. Just tried with master and that builds:
|
@ashthespy, that's what I've used as well. We should add proper tests. |
@c4milo any chance you could publish a new version? Node v14 is the current LTS, and would be nice not to have to depend on git version for this :-) |
I'm trying to install inotify using
But I get this error:
|
Will try to push a new version tomorrow, apologies for the delay! |
Thanks for your answers guys! When installing from master I get errors too, but it seems to install it. Could it be the version of npm, 6.14.12?
|
Getting similar errors as @lcofre.
|
Same error here... Either that or just deprecate the whole project (in npmjs too). |
I'm also having this on node v16.11.1, npm 8.0.0, node-gyp v8.2.0. This workaround worked for me:
|
It looks like the fix is simply to update bindings to 1.5.0. Simply updating that in the package.json and it now installs fine without needing to inject the rebuild. |
Sorry, what exactly does that mean? Can you tell me what file to edit, and how to edit it? |
I am getting a similar error when doing P.S.
|
I was first getting this error when I was trying to install gulp globally via However, I am now getting the same error when I run
|
Tried to use npm install inotify
[email protected]
[email protected]
[email protected]
The text was updated successfully, but these errors were encountered: