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

Upgrade breaks pbxproj file #12432

Closed
simonmitchell opened this issue Feb 17, 2017 · 2 comments
Closed

Upgrade breaks pbxproj file #12432

simonmitchell opened this issue Feb 17, 2017 · 2 comments
Labels
Resolution: Locked This issue was locked by the bot.

Comments

@simonmitchell
Copy link

Description

react-native upgrade breaks pbxproj file if project name has a space in it. I've just run react-native upgrade on a project, taken your changes for the project file and it has broken my pbxproj file.

Reproduction

Set the project's name to a name with a space in it. Run react-native upgrade from 0.35.0 -> 0.40.0

Solution

Do not strip parentheses around "path" property in pbxproj items if the project name has a space in it.

Additional Information

  • React Native version: Upgrading 0.35.0 -> 0.40.0
  • Platform: iOS
  • Operating System: MacOS
@simonmitchell
Copy link
Author

Same goes for Test file, it created it like so:

@interface Pixie MerchantTests : XCTestCase

@end

@implementation Pixie MerchantTests

@ericvicenti
Copy link
Contributor

Yeah, unfortunately you are responsible for merging this manually. The new tool react-native-git-upgrade may help make the upgrade changes more clear.

@facebook facebook locked as resolved and limited conversation to collaborators May 24, 2018
@react-native-bot react-native-bot added the Resolution: Locked This issue was locked by the bot. label Jul 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Resolution: Locked This issue was locked by the bot.
Projects
None yet
Development

No branches or pull requests

3 participants