-
Notifications
You must be signed in to change notification settings - Fork 204
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
npm install on create-daml-app fails with npm=9.3.1, nodejs=18.14.0 #16258
Comments
This is also a problem on Windows (and our Windows ad-hoc machines). NPM version: 9.3.1 |
|
It is indeed likely npm 9 that is at fault. When downgrading to the things that come with the 10.3.0 release: NPM version: 8.19.3 Everything works |
This seems relevant: nodejs/node#46542. Running
works for me on both macOS and Windows. From what I can gather this used to be the default but they changed it in (Node) 18.14; there seems to be a chance they'll reverse that, so I'm not sure what the appropriate action is on our side. Updating the instructions seems like a good approach as it should be harmless either way (i.e. whichever default they end up sticking with, explicitly setting the value should work). |
There is a lot of great daml examples out there but they are hard to run due to nodejs version issues. |
well, |
I don't have a Linux environment handy right now, but that shouldn't be the case. Can you double-check you have run |
Yes I did, it ends up with the same error as in #16258 (comment) One time I worked around that issue manually by commenting out daml.js dep and installing it afterwards but then I got this on
I've played with a bunch of daml projects and they all worked fine on Node version |
It looks like it got fixed on the node side - at least for me the GSG now works out of the box (i.e. without having to add the |
Closing as this is fixed as far as I can tell. Should anyone encounter this issue again please open a new ticket for it. |
Running npm install on create-daml-app (after generating the files is) fails because it cannot find a file:
I think it may be an NPM 8 vs NPM 9 issue.
You can repro with
The text was updated successfully, but these errors were encountered: