Skip to content

Make run-ios command play nicely with multiple Xcode versions - #19781

Closed
futuun wants to merge 1 commit into
react:masterfrom
futuun:master
Closed

Make run-ios command play nicely with multiple Xcode versions#19781
futuun wants to merge 1 commit into
react:masterfrom
futuun:master

Conversation

@futuun

@futuunfutuun commented Jun 17, 2018

Copy link
Copy Markdown
Contributor

I started using Xcode 10 beta and spotted that even though react-native cli is using cli tools from Xcode 10 for building it tries to open simulator from Xcode 9.4. Turned out that path to simulator app is hardcoded and doesn’t care about active developer directory.

Test Plan

  1. You have to have more than one Xcode version (ie. 9.4 and 10 beta).
  2. Change active developer directory to Xcode beta (xcode-select -s /Applications/Xcode-beta.app/Contents/Developer).
  3. Without fix react-native run-ios will open simulator from Xcode 9.4, with fix it'll open simulator from active developer directory.

Release Notes

[CLI][IOS] [BUGFIX] [./local-cli] - use simulator from active developer directory

@facebook-github-botfacebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jun 17, 2018
@futuunfutuun changed the title Make run-ios command play nicely with multiple Xcode versionsMake run-ios command play nicely with multiple Xcode versionsJun 17, 2018
@pull-bot

Copy link
Copy Markdown
Warnings
⚠️

📋 Release Notes - This PR may have incorrectly formatted Release Notes.

Generated by 🚫 dangerJS

@elicwhite

Copy link
Copy Markdown
Contributor

This is great. I actually just ran into this snag myself.

@facebook-github-botfacebook-github-bot added the Import Started This pull request has been imported. This does not imply the PR has been approved. label Jun 18, 2018

@facebook-github-botfacebook-github-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@TheSavior is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

hramos pushed a commit that referenced this pull request Jul 4, 2018
Summary:
I started using Xcode 10 beta and spotted that even though react-native cli is using cli tools from Xcode 10 for building it tries to open simulator from Xcode 9.4. Turned out that path to simulator app is hardcoded and doesn’t care about active developer directory.
1. You have to have more than one Xcode version (ie. 9.4 and 10 beta).
2. Change active developer directory to Xcode beta (`xcode-select -s /Applications/Xcode-beta.app/Contents/Developer`).
3. Without fix `react-native run-ios` will open simulator from Xcode 9.4, with fix it'll open simulator from active developer directory.
[CLI][IOS] [BUGFIX] [./local-cli] - use simulator from active developer directory
Closes#19781
Differential Revision: D8475915
Pulled By: TheSavior
fbshipit-source-id: d607f1f9eb9e565d4a017c21de50b5e569d171f9
@hramoshramos added this to the 0.58 RC milestone Sep 25, 2018
grabbou pushed a commit to react-native-community/cli that referenced this pull request Sep 26, 2018
Summary:
I started using Xcode 10 beta and spotted that even though react-native cli is using cli tools from Xcode 10 for building it tries to open simulator from Xcode 9.4. Turned out that path to simulator app is hardcoded and doesn’t care about active developer directory.
1. You have to have more than one Xcode version (ie. 9.4 and 10 beta).
2. Change active developer directory to Xcode beta (`xcode-select -s /Applications/Xcode-beta.app/Contents/Developer`).
3. Without fix `react-native run-ios` will open simulator from Xcode 9.4, with fix it'll open simulator from active developer directory.
[CLI][IOS] [BUGFIX] [./local-cli] - use simulator from active developer directory
Closesreact/react-native#19781
Differential Revision: D8475915
Pulled By: TheSavior
fbshipit-source-id: d607f1f9eb9e565d4a017c21de50b5e569d171f9
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BugCLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.Import StartedThis pull request has been imported. This does not imply the PR has been approved.Platform: iOSiOS applications.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@futuun@pull-bot@elicwhite@facebook-github-bot@hramos@react-native-bot