-
Notifications
You must be signed in to change notification settings - Fork 44
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
/usr/bin/env: ‘node\r’: No such file or directory ... #58
Comments
Thanks! |
In order to reproduce the issue you could:
Here is the result:
/usr/bin/env: ‘node\r’: No such file or directory npm ERR! A complete log of this run can be found in: |
npm publish from Windows handles CRLF incorrectly |
Alright, I'm trying that, but I'm getting errors with that package.json format - actually, using double quotes inside a double-quoted string isn't valid JSON, I'm using this package.json and it throws an error, first of all: package.json:
error:
Would you please be so kind to provide OS version, NPM and Node version so I can replicate? in OSX it's not working, I'm on Sierra 10.12.5, Node is 8.1.1 and NPM is 5.0.3. Also, the publish was made under this same OSX machine, so that bug doesn't apply to me :P |
You need to escape the double quotes: "watch:dev": "parallelshell \"npm run build\" \"npm run serve\"" Running on: |
Alright, you were right! I managed to reproduce it, I'll fix it right away, thanks for reporting :D |
You are very welcome |
Fixed & published! 👍 |
Thank You ! |
Two things:
If you are working on Linux/OS X you will have the following issue:
/usr/bin/env: ‘node\r’: No such file or directory ...
This is bc file is saved with line separator CRLF instead LF, just save the index.js using LF line separator will solve the issue
index.js on version 3.0.0 is the same as the previous version, it does not has any change, still cwd: process.cwd, instead cwd: process.cwd(). Developer made the change on test/index.coffee but never on index.js file
The text was updated successfully, but these errors were encountered: