-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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
Parse simulator list #23293
Parse simulator list #23293
Conversation
Generated by 🚫 dangerJS |
As master obviously is not in sync with latest stable, I made this pull request towards 0.58-stable. Should probably be towards the lastest development branch, but it was not obvious to me which one that should be. Feel free to change the target of the PR. |
For reference.. Here is what the output from
|
Could you please backport to |
@jmatsushita we have a separate repo for discussing backports and cherry picks over at http://github.com/react-native-community/react-native-releases |
Parse simulator list
Summary
When parsing simulator names in order to run on iOS simulator, the name now contains the prefix "com.apple.CoreSimulator.SimRuntime." Therefore the check to filter out simulators that are not iOS or tvOS needs not to check for startsWith, but rather to check for includes.
Without the fix, I am not able to start and run the iOS simulator through the react-native cli
This started happening for me after I installed the latest XCode (beta). I am currently using XCode "Version 10.2 beta (10P82s)", but I have tried using xcode-select to point towards the latest stable XCode version as well, but the problem still occurs. My version of xcrun is "xcrun version 43.1."
Changelog
[iOS] [Changed]- Enabling start of iOS/tvOS simulators when simulator versions are reported with apple-prefixes.
Test Plan
Run the command react-native run-ios. Without the fix I get the error
Could not find iPhone X simulator
With the fix, it starts fine.