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

Workaround to make yarn ios work with Xcode 10 #228

Merged
merged 1 commit into from
Nov 8, 2018
Merged

Workaround to make yarn ios work with Xcode 10 #228

merged 1 commit into from
Nov 8, 2018

Conversation

koke
Copy link
Member

@koke koke commented Nov 8, 2018

It seems the issue is that Xcode is not automatically downloading required dependencies, so this adds a preios step to download them before trying to build the app.

See facebook/react-native#20774

I don't have Xcode 9 installed, but I believe it should not affect building with 9.

It seems the issue is that Xcode is not automatically downloading required
dependencies, so this adds a preios step to download them before trying to build
the app.

See facebook/react-native#20774
@koke koke requested review from pinarol and etoledom November 8, 2018 07:33
@hypest
Copy link
Contributor

hypest commented Nov 8, 2018

FWIW, I tried this PR with Xcode 9 and worked fine.

$ xcodebuild -version
Xcode 9.4.1
Build version 9F2000

Copy link
Contributor

@pinarol pinarol left a comment

Choose a reason for hiding this comment

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

Looks good.

Test steps:

set Xcode10 as default
rm -rf node_modules
yarn cache clean
yarn install
yarn start
yarn ios

some errors produced on the console and the screen got stuck as follows:

screen shot 2018-11-08 at 12 28 59

Then I tried the same including the fix and everything worked ok.

@koke koke merged commit 16c6a60 into master Nov 8, 2018
@koke koke deleted the xcode10 branch November 8, 2018 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants