-
Notifications
You must be signed in to change notification settings - Fork 244
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
Migrate from GYP to GN #330
Comments
@deepak1556 has node moved to GN? |
Nope and they don't plan to do that anytime soon nodejs/node#21410, nodejs/TSC#648 , nodejs/TSC#642 Here is some discussion about getting python3 support in node-gyp nodejs/node-gyp#1337 and there are workarounds mentioned in that thread. I would recommend not to change the build system, its more technical debt when the adoption rate is nearly zero in the community. But some modules rely on https://github.com/cmake-js/cmake-js which is looked as an alternative. |
That all makes sense. I'm sorry I didn't know more. We've just had a hell of a time with gyp and python2.7 especially with |
We have a ton of issues in
script-runner
because ofnode-pty
build failing due to node-gyp not being compatible with python3.e.g. ioquatix/script-runner#103
It looks like the latest version of Electron moved to GN: https://electronjs.org/blog/gn
Maybe it would be a good idea for this project.
The text was updated successfully, but these errors were encountered: