Skip to content
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

iOS build fails if path of a project path includes a space #22354

Closed
3 tasks done
owinter86 opened this issue Nov 20, 2018 · 11 comments
Closed
3 tasks done

iOS build fails if path of a project path includes a space #22354

owinter86 opened this issue Nov 20, 2018 · 11 comments
Labels
Bug Impact: Regression Describes a behavior that used to work on a prior release, but stopped working recently. 📮Known Issues This indicates an issue that refers to a bug or limitation of RN that is not currently being handled Platform: iOS iOS applications. Priority: Low Resolution: Locked This issue was locked by the bot.

Comments

@owinter86
Copy link

Environment

React Native Environment Info:
System:
OS: macOS 10.14.1
CPU: (4) x64 Intel(R) Core(TM) i5-7267U CPU @ 3.10GHz
Memory: 317.20 MB / 16.00 GB
Shell: 5.3 - /bin/zsh
Binaries:
Node: 10.13.0 - ~/.nvm/versions/node/v10.13.0/bin/node
Yarn: 1.12.3 - /usr/local/bin/yarn
npm: 6.4.1 - ~/.nvm/versions/node/v10.13.0/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
SDKs:
iOS SDK:
Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
Android SDK:
API Levels: 26, 27, 28
Build Tools: 26.0.3, 27.0.3, 28.0.2, 28.0.3
System Images: android-26 | Google APIs Intel x86 Atom_64, android-28 | Google APIs Intel x86 Atom
IDEs:
Android Studio: 3.1 AI-173.4907809
Xcode: 10.1/10B61 - /usr/bin/xcodebuild
npmPackages:
react: 16.6.1 => 16.6.1
react-native: 0.57.5 => 0.57.5
npmGlobalPackages:
react-native-cli: 2.0.1
react-native-git-upgrade: 0.2.7

Description

If any folder within your project directory contains a space the build will fail, a new react native project was created in folder untitled and built with no issues, and another created in folder untitled folder and failed to find a file within the directory.

../Documents/untitled folder/test/node_modules/react-native/scripts/ios-install-third-party.sh: line 56: /ios-configure-glog.sh: No such file or directory

Reproducible Demo

This is reproducible if you create a new RN project in a folder with a space.

@radko93
Copy link
Contributor

radko93 commented Nov 21, 2018

@owinter86 can you try react-native init projectname --version [email protected] and let me know if this also happens?

@owinter86
Copy link
Author

@radko93 I can confirm that this issue was introduced in 0.57.5 and this does not occur on 0.57.4.

@radko93
Copy link
Contributor

radko93 commented Nov 22, 2018

@kelset I think it's a regression from the Xcode fix

@kelset kelset added the Impact: Regression Describes a behavior that used to work on a prior release, but stopped working recently. label Nov 22, 2018
@kelset
Copy link
Contributor

kelset commented Nov 22, 2018

uhm yeah it looks like it :/

But i can't really understand why 🤔

cc @mmccartney

BTW does the CLI allow the creation of projectNames with empty spaces...?

@kelset kelset added the Platform: iOS iOS applications. label Nov 22, 2018
@radko93
Copy link
Contributor

radko93 commented Nov 22, 2018

I also couldn't really get why but that's the only change that's related.
@kelset it's not about the project name, it's about the path.

@kelset
Copy link
Contributor

kelset commented Nov 22, 2018

Probably related / could give context: wix/Detox#894

@radko93
Copy link
Contributor

radko93 commented Nov 22, 2018

@kelset probably not related as we don't use node for running script.
Running:

cd node_modules/react-native && scripts/ios-install-third-party.sh && cd third-party && cd glog-0.3.5 && ./configure && cd ../../../../

works with any path so it has to be something with how xcode runs the script.
It might be here but maybe @mmccartney will know more about it

@radko93 radko93 changed the title Build fails if a folder name has a space in the project directory. iOS build fails if a a project path includes a space Nov 22, 2018
@radko93 radko93 changed the title iOS build fails if a a project path includes a space iOS build fails if path of a project path includes a space Nov 22, 2018
@kelset kelset added the 📮Known Issues This indicates an issue that refers to a bug or limitation of RN that is not currently being handled label Nov 22, 2018
@hramos hramos removed the Bug Report label Feb 6, 2019
@thymikee
Copy link
Contributor

Hello there 👋 by using the repro / the list of steps to reproduce, we can't actually see the reported bug on RN 0.59 so we'll close this issue. It may have something to do with your local setup, please double check that you are using the correct version of Android Studio / XCode / Android NDK / Yarn / etc.

@canpoyrazoglu
Copy link

canpoyrazoglu commented May 15, 2019

Same issue here on RN 0.59. Lost about 5 hours because of this bug. I've removed the space from my project's path and everything worked perfectly.

@Bersaelor
Copy link

Bersaelor commented Jun 25, 2019

Just wasted another 2h on this exact same issue.
"react-native": "0.59.5",, Xcode 10.2.1
@thymikee

@thymikee
Copy link
Contributor

@Bersaelor @canpoyrazoglu I think I stated clearly that I couldn't reproduce the issue. If you still face it, it's likely because of some environmental differences. Without that information, I'm unable to help more. Please provide clear reproduction steps (repo, env info etc) on a smallest possible example and if you're sure it's a bug somewhere in RN or CLI, file a new issue accordingly. Otherwise it's not actionable.

@facebook facebook locked as resolved and limited conversation to collaborators Mar 19, 2020
@react-native-bot react-native-bot added the Resolution: Locked This issue was locked by the bot. label Mar 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug Impact: Regression Describes a behavior that used to work on a prior release, but stopped working recently. 📮Known Issues This indicates an issue that refers to a bug or limitation of RN that is not currently being handled Platform: iOS iOS applications. Priority: Low Resolution: Locked This issue was locked by the bot.
Projects
None yet
Development

No branches or pull requests

8 participants