-
-
Notifications
You must be signed in to change notification settings - Fork 120
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
Node 15 upgrades npm to 7, which breaks prefix installs #123
Comments
Can confirm it's an issue with node 15:
If I create Honestly I think this is something |
I have the same issue. Works when I set npmVersion to 6.14.8 but not when setting to 7.0.8. I am using release 2.2.4. |
This is fixed in |
for me it is breaking even with
|
Also am I right in thinking that |
That's correct, the name for the But the |
Thanks @deepy Perhaps it is worth putting the release notes for 3.0 that the name of the task has changed, it is a breaking change. |
Oh it hasn't changed, when yarn support was initially added 4 years ago it was with the same name, but I do know that a lot of people previously missed But I realize now that when I added a clarification in the documentation I forgot to account for the same issue with |
I've tried to reproduce this, but on rc4 I can't get the
running
|
Sorry for the slow response. Been busy. Here is the file that is failing for me with Gradle 6.7.1 on MacOS:
|
(venv) ➜ temp gradle yarn
npm ERR! A complete log of this run can be found in:
FAILURE: Build failed with an exception.
BUILD FAILED in 13s |
Oh this is sneaky, not sure how it worked on my machine but the directory being created is |
You probably already had that directory when you ran it. |
3.0.0-rc4 - kotlin rewrite, lazy configuration, api changes, proxy fixes. * Plugin rewritten in Kotlin (issue node-gradle#17) (thanks mikejhill for the pull request) * Improved Kotlin DSL support * Upgraded default Node.js version to 12.18.1 (bundled with npm 6.14.5), the latest LTS version * Lazy configuration support * Fix some remaining input/output declaration issues (issue node-gradle#34) * Gradle 5.6.4+ support (instead of Gradle 5.0.0+ before) * Node.js 10+ support (issue node-gradle#100) * Use http:// for both HTTP and HTTPS_PROXY * No longer configures proxy if there's already proxy settings present * Support for npm 7+ node-gradle#123 See CHANGELOG for full details and upgrade instructions.
I've created #127 which should fix this and there's tests that prove it this time 😅 |
@deepy Thanks, much appreciated. Let me know when it has been released so I can test it as well. |
If you clone the branch you can test it in your project with |
@deepy I just tested it as you described and that branch is working for me. Thank you. |
Great! I'll merge and create a release this evening so by tomorrow it should be fixed :-) |
3.0.0-rc5 is out now |
When I set Node.js version
15.0.1
to be used, theyarnSetup
task fails like so:Using version
14.14.0
works fine. Any ideas?The text was updated successfully, but these errors were encountered: