-
Notifications
You must be signed in to change notification settings - Fork 153
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
Fix Problem with M1 Macs #30
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple of requests but overall looks good, thanks!
packages/mobile/package.json
Outdated
@@ -5,6 +5,7 @@ | |||
"scripts": { | |||
"android": "react-native run-android", | |||
"ios": "react-native run-ios", | |||
"ipad": "react-native run-ios --simulator='iPad Air (4th generation)'", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we avoid adding this script? You can run it anyway passing the simulator options to yarn ios
👍
package.json
Outdated
@@ -25,6 +25,7 @@ | |||
"android:studio": "yarn workspace @my-app/mobile studio", | |||
"ios:metro": "yarn workspace @my-app/mobile start", | |||
"ios:start": "yarn workspace @my-app/mobile ios", | |||
"ipad:start": "yarn workspace @my-app/mobile ipad", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we avoid adding this script? You can run it anyway passing the simulator options to yarn ios
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sorry about that one, I was using it on my project but somehow I made a pull request for that change, I submitted a new commit without it
This reverts commit e6cc3cc.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
For Apple Silicon to work at all you need to address a linker problem with react-native and Swift libraries
Fix as described in:
facebookarchive/react-native-fbsdk#755