Skip to content
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

Fails to install/compile under node v12.10.0 #30

Open
gk1 opened this issue Sep 11, 2019 · 7 comments
Open

Fails to install/compile under node v12.10.0 #30

gk1 opened this issue Sep 11, 2019 · 7 comments

Comments

@gk1
Copy link

gk1 commented Sep 11, 2019

udev-install.txt

@cheery
Copy link
Owner

cheery commented Sep 11, 2019 via email

@jacobq
Copy link

jacobq commented Oct 14, 2019

@cheery, wow, and it looks like you're even using nan which is supposed to make those compatibility nightmares mostly go away...

Would you mind updating the engines section of your package.json so it's clear what versions of Node this package actually does support? It looks like it works with 10.x but not with 12.x, which is scheduled to become LTS in about a week.

@mortie
Copy link

mortie commented Oct 14, 2019

In 2017, after #19, I decided to just write my own udev library, where the udev part is a tiny C program which emits JSON on stdout and accepts commands on stdin, with some javascript to wrap it: https://github.com/mortie/dedaemon/tree/master/udev

It's not documented, and probably not feature complete, and I never put it up as its own package, but it has just sat there and worked without being touched for those two and a half years. As long as core node.js libraries and the udev C library doesn't change, it'll work for many more. Might be something to consider.

I've used Google C++ libraries before. Nothing they actively work on is ever anywhere near stable. As long as you're using stuff from the V8 namespace, I imagine breakages will continue until the node.js team stops upgrading to new versions of V8.

Also, compare http://npm.broofa.com/?q=udev and http://npm.broofa.com/?q=dedaemon ;)

@lucamarogna
Copy link

Try this fork: https://github.com/ConnectionSystems/node-udev
I think it worths a look because it actually works with NodeJS 12 and perphaps you can integrate his Pull Request into your project.

Just reach out to him if you would to.

@jacobq
Copy link

jacobq commented Jan 16, 2020

@donlucacorleone thanks for pointing out ConnectionSystems@39957a9 (though I don't see any open PR)

@miteshdube
Copy link

@donlucacorleone Thank you for this repository it solved my problem.

@miteshdube
Copy link

Can anyone tell me where i'll get API for node-udev?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants