Closed
Description
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
Activity
darkguy2008 commentedon Jun 15, 2017
I think you need to change your git CRLF settings to Auto, depending on your OS. I am using OSX and I didn't run into that issue. Any pointers on how to reproduce it? I'm using SourceTree to do the Git work.
It is because it's not being used anywhere as far as I know. index.coffee calls it, but the tests don't do anything whether if I change it to cwd or cwd() so I didn't touch it. Do you have any screenshot to prove the issue so I can see?
Thanks!
wvillegasm commentedon Jun 15, 2017
In order to reproduce the issue you could:
npm i -D parallelshell
Write your script into the package.json, for instance:
"scripts": {
"build": "webpack -w",
"serve": "browser-sync start --server --startPath \"./built/static/\" --serveStatic \"./built/static/\" --index \"index.html\" --files \"./built/static/**/*.+(html|css|js)\" --port 3001",
"watch:dev": "parallelshell "npm run build" "npm run serve""
}
Run your script:
npm run watch:dev
Here is the result:
/usr/bin/env: ‘node\r’: No such file or directory
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn
npm ERR! demo@1.0.0 watch:dev:
parallelshell "npm run build" "npm run serve"
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the demo@1.0.0 watch:dev script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
wvillegasm commentedon Jun 15, 2017
npm publish from Windows handles CRLF incorrectly
npm/npm#2097
darkguy2008 commentedon Jun 15, 2017
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
wvillegasm commentedon Jun 15, 2017
You need to escape the double quotes:
"watch:dev": "parallelshell \"npm run build\" \"npm run serve\""
Running on:
Linux 4.10.0-20-generic #22-Ubuntu
node -v ~ v8.0.0
npm -v ~ 5.0.3
darkguy2008 commentedon Jun 15, 2017
Alright, you were right! I managed to reproduce it, I'll fix it right away, thanks for reporting :D
wvillegasm commentedon Jun 15, 2017
You are very welcome
Fix issue #58 (CRLF line endings) & index.js cwd(), thanks @wvillegasm !
darkguy2008 commentedon Jun 15, 2017
Fixed & published! 👍
wvillegasm commentedon Jun 15, 2017
Thank You !
It is working like a charm
👯♂️
1 remaining item