-
Notifications
You must be signed in to change notification settings - Fork 288
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
Download and link node.lib
on Windows
#583
Download and link node.lib
on Windows
#583
Conversation
I think this now does all the basics. You can pass in a node.lib path using |
@goto-bus-stop thanks for this PR! I'm trying to add x86 support to Signal Desktop (signalapp/ringrtc#12) but was running into Rust compiler issues:
I'm assuming this PR will solve that as you're adding support for x86 as far as I can see 😊 If I can help in any way to get this PR pushed through for a new release, please let me know! Thanks in advance 🚀 |
Yes, note that this PR is only for the N-API runtime, behind the This PR is currently only waiting for another review round by a core team member :) |
This enables N-API tests on Windows and sets up the linking stuff that we need for the average Node.js addon.
Todo:
This also does not yet handle the case where the
node.exe
executable has been renamed, and will crash if loaded into anotnode.exe
. That will be a separate PR.