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

[templates] fix out of sync template from react-native 0.68 #17266

Merged

Conversation

Kudo
Copy link
Contributor

@Kudo Kudo commented Apr 28, 2022

Why

__apply_Xcode_12_5_M1_post_install_workaround was missing when i did #15914. that may introduced some build issue on m1 machine when flipper is enabled.

How

Test Plan

tested on m1 macos

$ EXPO_BETA=1 expo init sdk45 # select bare
# update ios/Podfile with the change and uncomment use_flipper!()
$ expo run:ios
# test with flipper

Checklist

  • n/a Documentation is up to date to reflect these changes (eg: https://docs.expo.dev and README.md).
  • This diff will work correctly for expo build (eg: updated @expo/xdl).
  • This diff will work correctly for expo prebuild & EAS Build (eg: updated a module plugin).

@expo-bot expo-bot added the bot: passed checks ExpoBot has nothing to complain about label Apr 28, 2022
@Kudo Kudo merged commit f1c4eb1 into main Apr 28, 2022
@Kudo Kudo deleted the @kudo/sdk45/__apply_Xcode_12_5_M1_post_install_workaround branch April 28, 2022 20:00
brentvatne pushed a commit that referenced this pull request Apr 28, 2022
…#17266)

# Why

`__apply_Xcode_12_5_M1_post_install_workaround` was missing when i did #15914. that may introduced some build issue on m1 machine when flipper is enabled.

# How

- add missing `__apply_Xcode_12_5_M1_post_install_workaround` as from [react-native 0.68 template](https://github.com/facebook/react-native/blob/b5f1b26a8ff4014e45bbb95c5956ba2c2cf286db/template/ios/Podfile#L35)
- also update `use_flipper!()` statement as from [react-native 0.68 template](https://github.com/facebook/react-native/blob/b5f1b26a8ff4014e45bbb95c5956ba2c2cf286db/template/ios/Podfile#L31). i think the old flipper issue should be already resolved.

# Test Plan

tested on m1 macos

```sh
$ EXPO_BETA=1 expo init sdk45 # select bare
# update ios/Podfile with the change and uncomment use_flipper!()
$ expo run:ios
# test with flipper
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot: passed checks ExpoBot has nothing to complain about
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants