-
-
Notifications
You must be signed in to change notification settings - Fork 349
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
10.0.0-beta.45+ npm install fails with node-gyp v8.1.0 on windows #741
Comments
hmm interesting, it could be. Our Windows builds seem to be passing, but 🤷 . By any chance, do any of the latter versions work (e.g. beta.49?) |
I tried beta.45-49 but none of those seem to work. Only beta.44 works. Maybe this is also a local problem. On our jenkins build machine we had to install gcc-c++ compiler to be able to build it again. |
Thanks for the report. It's strange, because we have windows builds running fine in CI. I think you're right that this is a regression in node-gyp. Since node-gyp is bundled with npm, it's not super clear what we can do to fix it, other than wait for the next version of npm to be released again (I think?). I'm keen to get to the bottom of why this is happening for you but not in CI - what version of npm are you using? Do you know if you have changed the version of node-gyp from the bundled one? Our windows builds use npm versions 6.14.12 and 6.14.15 (depending on the node version). For node 14, we use the same version you are using, v14.17.6. Hmm. |
We are using the latest version of pnpm and on our CI Server (which is a SUSE Linux Enterprise) we had to install |
I finally managed to solve the "problem" by deleting the local pnpm store and build everything from scratch again. This is not a problem of pact-js. Sorry for the circumstances! |
Glad it's sorted! Thanks for the update! |
Thanks @schowave - did you still need to have @TimothyJones this is a good reminder. Perhaps we should have a "compatibility" table or something with these sorts of things (e.g. yarn2, npm, pnpm)? |
That’s a good idea. This one will go away, I think. It’s a regression in node gyp 8.1.0 which looks like it is fixed in 8.2.0 |
We should link to the windows instructions for node-gyp, though. Since it won’t work without node-gyp |
We were using an older version of pnpm (6.7.1). With the current version 6.18.0 everything works fine again. |
Thanks for clarifying. |
Software versions
Please provide at least OS and version of pact-js
Issue Checklist
Please confirm the following:
Expected behaviour
npm install is possible
Actual behaviour
npm install is not possible
Steps to reproduce
How can someone else reproduce this bug?
Use @pact-foundation/pact 10.0.0-beta.45 in your project as devDependency and try to run
npm install
.Current workaround for me is to execute
npm i -D [email protected]
beforehand, and then the npm install works again.Does this have something to do with nodejs/node-gyp#2399 or nodejs/gyp-next#121 ?
Relevant log files
The text was updated successfully, but these errors were encountered: