-
Notifications
You must be signed in to change notification settings - Fork 980
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
Use nan to be compatible with newer Node.js versions. #1
Comments
@octalmage, robotjs is awesome. I want to use this with latest node.js. When will this be ready? |
I'm glad you like RobotJS! Unfortunately I'm not sure when it will be ready. It was harder than I originally thought and I need time to work on it. It is something I'm very interested in though. RobotJS is almost useless to me without this. |
I am also trying to modify RobotJS using nan and it is harder than I imagined. Thanks for your response. 👍 |
No problem! I started the work here: https://github.com/octalmage/robotjs/tree/nan I'll take a look now, see if I can make some progress. |
Actually got it working and converted a function: I'll finish the rest later, shouldn't be much longer. |
Hey thanks, that function got build and working perfectly. |
Great work in 09b115e! Do you want to finish off the conversion? |
At first I just compiled that function then I made some changes to other functions by forking the project 09b115e. And all of it is compiled in the node version 0.11.13 and woking. |
I saw that, thanks a ton man! When it's done submit a pull request! 😉 🎉 |
@anand-seeenivasagam since your repository now contains code that is not cross platform it can't be merged. I finished the conversion myself. |
Thats allright! I will try to make the changes cross platform because we need it in future. Thanks! :) |
Awesome! Thanks again for your help! |
Node.js v12
I need to use nan to make RobotJS compatible with newer versions of node.js.
The text was updated successfully, but these errors were encountered: