Skip to content

Update node-pre-gyp to fix Node 12/14 installation problems in some environments#5991

Merged
danpat merged 1 commit intomasterfrom
danpat_fix_node_pre_gyp
Mar 23, 2021
Merged

Update node-pre-gyp to fix Node 12/14 installation problems in some environments#5991
danpat merged 1 commit intomasterfrom
danpat_fix_node_pre_gyp

Conversation

@danpat
Copy link
Copy Markdown
Member

@danpat danpat commented Mar 23, 2021

Issue

needle is a HTTP client library that node-pre-gyp uses to download binaries. The version of node-pre-gyp we depended on (0.12.0) depended on an old version of needle that has this bug: tomas/needle#312 - it broke with Node 12.

For users with the request module available, this problem wasn't noticed. But in greenfield installations of npm install osrm, the following error was being encountered:

> [email protected] install /project/node_modules/osrm
> node-pre-gyp install --fallback-to-build=false || ./scripts/node_install.sh

node-pre-gyp WARN Using needle for node-pre-gyp https download
node-pre-gyp ERR! install error
node-pre-gyp ERR! stack Error: bad download

Updating node-pre-gyp upgrades the needle dependency, and fixes the above problem.

Tasklist

@danpat danpat merged commit 5740604 into master Mar 23, 2021
@DennisOSRM DennisOSRM deleted the danpat_fix_node_pre_gyp branch November 6, 2022 14:12
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

Successfully merging this pull request may close these issues.

2 participants