Repository navigation
e2e fails when the path has space #1652
Description
If you are reporting a bug, please fill in below. Otherwise feel free to remove this template entirely.
Can you reproduce the problem with latest npm?
Yes
Description
e2e fails when pwd has space in its path. eg /Users/adeviankakrisnafadlil/Local Sites/create-react-app
What are you reporting?
bug
Expected behavior
It should run the e2e test
Actual behavior
It fails with this error:
+ cd /Users/adeviankakrisnafadlil/Local Sites/create-react-app/packages/create-react-app
tasks/e2e-simple.sh: line 115: cd: /Users/adeviankakrisnafadlil/Local: No such file or directory
++ set +x
e2e-simple.sh: ERROR! An error was encountered executing line 115.
Cleaning up.
tasks/e2e-simple.sh: line 24: cd: /Users/adeviankakrisnafadlil/Local: No such file or directory
Exiting with error.
Environment
Run these commands in the project folder and fill in their results:
npm ls react-scripts(if you haven’t ejected): 9.2.0node -v: v7.4.0npm -v: 4.0.5
Then, specify:
- Operating system: macOs Sierra
- Browser and version: n/a
Reproducible Demo
Just rename your current create-react-app repo with create react-app or rename the parent folder to have space in it
Activity
Timer commentedon Feb 26, 2017
Would you like to submit a pull request fixing this? 😄
viankakrisna commentedon Feb 26, 2017
Haven't found the fix yet T_T
Will dig more tomorrow, in the meantime maybe someone with power in shell scripting could help? 😄
Timer commentedon Feb 26, 2017
I believe we just need to wrap usages of cd (and related commands) with
".e.g.
cd $root_path/packages/react-scripts->cd "$root_path/packages/react-scripts"viankakrisna commentedon Feb 26, 2017
ok then, i thought just wrapping PWD with it would be sufficient, will do it tomorrow, it's quite a lot and haven't got a good sleep today 😄
Timer commentedon Mar 2, 2017
@viankakrisna will you take a look at this soon? 😄