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

fix: recommended node-gyp version in node.h error #37942

Merged

Commits on Apr 12, 2023

  1. fix: recommended node-gyp version in node.h error (electron#37829)

    fix: recommended node-gyp version in node.h error
    
    In
    https://github.com/electron/electron/blob/main/docs/tutorial/using-native-node-modules.md#using-npm,
    we recommend setting the `npm_config_disturl` variable but doing that
    does not work on node-gyp v8.4.0 because after
    nodejs/node-gyp#2497
    landed, the dist URL was read only from `gyp.opts['dist-url']`. The fix
    for reading the value from `npm_config_disturl` by parsing
    `gyp.opts.disturl` was landed in
    nodejs/node-gyp#2547 and that change was
    released in node-gyp v9.0.0, so this change updates the error macro to
    recommend node-gyp v9.0.0 as the minimum required version.
    
    Signed-off-by: Darshan Sen <raisinten@gmail.com>
    RaisinTen committed Apr 12, 2023
    Configuration menu
    Copy the full SHA
    c421eda View commit details
    Browse the repository at this point in the history