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

[Fabric] Use RCT_NEW_ARCH_ENABLED flag for enabling new arch #1782

Merged
merged 3 commits into from
Apr 11, 2023

Conversation

shwanton
Copy link

Please select one of the following

  • I am removing an existing difference between facebook/react-native and microsoft/react-native-macos 👍
  • I am cherry-picking a change from Facebook's react-native into microsoft/react-native-macos 👍
  • I am making a fix / change for the macOS implementation of react-native
  • I am making a change required for Microsoft usage of react-native

Summary

RCT_NEW_ARCH_ENABLED flag is what RN core uses for enabling the new architecture

Changelog

[GENERAL] [FIXED] - Use build flag from RN mobile for enabling new architecture

Test Plan

Fabric builds

RCT_NEW_ARCH_ENABLED=1 bundle exec pod install --verbose

CleanShot 2023-04-10 at 13 43 54

@shwanton shwanton changed the title [Fabric] Use Replace RCT_NEW_ARCH_ENABLED flag for enabling new arch [Fabric] Use RCT_NEW_ARCH_ENABLED flag for enabling new arch Apr 10, 2023
@shwanton shwanton mentioned this pull request Apr 10, 2023
4 tasks
@shwanton shwanton marked this pull request as ready for review April 10, 2023 20:54
@shwanton shwanton requested a review from a team as a code owner April 10, 2023 20:54
@Saadnajmi Saadnajmi merged commit 3dd054e into microsoft:main Apr 11, 2023
Saadnajmi pushed a commit to Saadnajmi/react-native-macos that referenced this pull request Apr 11, 2023
…ft#1782)

* Replace USE_FABRIC w/ RCT_NEW_ARCH_ENABLED flag for enabling new arch

* Native CXX TM builds for macOS

* Update RNTester podfile w/ correct flag

---------

Co-authored-by: Shawn Dempsey <[email protected]>
Saadnajmi added a commit that referenced this pull request Apr 11, 2023
…1783)

* Replace USE_FABRIC w/ RCT_NEW_ARCH_ENABLED flag for enabling new arch

* Native CXX TM builds for macOS

* Update RNTester podfile w/ correct flag

---------

Co-authored-by: Shawn Dempsey <[email protected]>
Co-authored-by: Shawn Dempsey <[email protected]>
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.

2 participants