-
Notifications
You must be signed in to change notification settings - Fork 24.4k
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 Cocoapods for Xcode 14.3.0 #36759
Conversation
This pull request was exported from Phabricator. Differential Revision: D44605617 |
Base commit: a2d2696 |
Summary: Pull Request resolved: facebook#36759 On Thursday the 30th, Apple Released Xcode 14.3.0. This Version of Xcode enforce some version checks for which React-Codegen, which supported iOS 11 as minimum supported version, could not be build anymore. This change ensue that React-Codegen is always aligned to the min version supported by React Native. Plus, it moves CircleCI's Xcode to 14.3.0, to keep this problem in Check. While working on this, I figured that, with the monorepo, Ruby tests stopped working because they were in the wrong folder: I moved them in the right one. ## Changelog: [iOS][Fixed] - Make React Native build with Xcode 14.3.0 and fix tests Differential Revision: D44605617 fbshipit-source-id: e634c70607767ef9a2af08848bce9407b66a3431
This pull request was exported from Phabricator. Differential Revision: D44605617 |
f0b4b7b
to
b731d33
Compare
@@ -527,7 +535,7 @@ def get_podspec_no_fabric_no_script | |||
'source' => { :git => '' }, | |||
'header_mappings_dir' => './', | |||
'platforms' => { | |||
'ios' => '11.0', | |||
'ios' => '12.4', |
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.
shouldn’t it be also min_ios_version_supported
?
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.
it depends on the philosophy you adopt to write tests.
If you use min_ios_version_supported
here, once we move the function to the proper file and we reuse the same function of production, this test would be tautological, as changing what the function in prod returns will also change what returned here.
Personally, I prefer to see my test failing if I bump a version in production code, so I can double check that the change was done on purpose and not by mistake. That's why I hardcoded the value.
Does it make sense?
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.
interesting – makes sense!
hey I'm new to developing. my react native project stopped working because of the xcode update. so my question is where do i look for updates? do i need to wait for an xcode update? Cocoapods update? I don't really understand what's happening. Thanks for your help! |
@LukeIKG you need to wait until this gets merged or apply the changes manually with patch or downgrade your Xcode to 14.2 |
Thanks for the quick response! |
So, the root cause is that one of our pods (React-Codegen) is supporting iOS 11, which does not build with Xcode 14.3. This change bumps the above mentioned pod, fixing the problem. When the patch is out, yes, you'll have to bump RN in the package.json, apply the changes suggested here and then Meanwhile, you can use the "patch" suggested here Or, I think, use Xcode 14.2 toolchain with Xcode 14.3, or downgrade Xcode to 14.2. |
Got it to work, yay! |
Summary: Pull Request resolved: facebook#36759 On Thursday the 30th, Apple Released Xcode 14.3.0. This Version of Xcode enforce some version checks for which React-Codegen, which supported iOS 11 as minimum supported version, could not be build anymore. This change ensue that React-Codegen is always aligned to the min version supported by React Native. Plus, it moves CircleCI's Xcode to 14.3.0, to keep this problem in Check. While working on this, I figured that, with the monorepo, Ruby tests stopped working because they were in the wrong folder: I moved them in the right one. ## Changelog: [iOS][Fixed] - Make React Native build with Xcode 14.3.0 and fix tests Reviewed By: blakef Differential Revision: D44605617 fbshipit-source-id: 182bad20fff2bc19f900000560152369013bb481
This pull request was exported from Phabricator. Differential Revision: D44605617 |
b731d33
to
ef27760
Compare
This pull request has been merged in cb88e75. |
For devs working with older versions of react-native (am working with 0.70.6) looking to apply the "patch", looks like we need to define min_ios_version_supported ourselves. The patch then looks like the following:
|
Summary: Pull Request resolved: #36759 On Thursday the 30th, Apple Released Xcode 14.3.0. This Version of Xcode enforce some version checks for which React-Codegen, which supported iOS 11 as minimum supported version, could not be build anymore. This change ensue that React-Codegen is always aligned to the min version supported by React Native. Plus, it moves CircleCI's Xcode to 14.3.0, to keep this problem in Check. While working on this, I figured that, with the monorepo, Ruby tests stopped working because they were in the wrong folder: I moved them in the right one. ## Changelog: [iOS][Fixed] - Make React Native build with Xcode 14.3.0 and fix tests Reviewed By: blakef Differential Revision: D44605617 fbshipit-source-id: 3ec1f5b36858ef07d9f713d74eb411a1edcccd45
This comment was marked as abuse.
This comment was marked as abuse.
I'm a little confused with this statement:
Xcode 14.3 can build for iOS 11, so I assume that there's some code in @cipolleschi could you provide more clarity on what is causing the build issues? I'm currently running an older RN version (0.67.4), which does have a minimum iOS target version of 11. I'd like to know if bumping it to 12.4 is the only viable solution right now to be able to build with Xcode 14.3 |
Summary: Pull Request resolved: facebook#36759 On Thursday the 30th, Apple Released Xcode 14.3.0. This Version of Xcode enforce some version checks for which React-Codegen, which supported iOS 11 as minimum supported version, could not be build anymore. This change ensue that React-Codegen is always aligned to the min version supported by React Native. Plus, it moves CircleCI's Xcode to 14.3.0, to keep this problem in Check. While working on this, I figured that, with the monorepo, Ruby tests stopped working because they were in the wrong folder: I moved them in the right one. ## Changelog: [iOS][Fixed] - Make React Native build with Xcode 14.3.0 and fix tests Reviewed By: blakef Differential Revision: D44605617 fbshipit-source-id: 3ec1f5b36858ef07d9f713d74eb411a1edcccd45
Summary: Pull Request resolved: facebook#36759 On Thursday the 30th, Apple Released Xcode 14.3.0. This Version of Xcode enforce some version checks for which React-Codegen, which supported iOS 11 as minimum supported version, could not be build anymore. This change ensue that React-Codegen is always aligned to the min version supported by React Native. Plus, it moves CircleCI's Xcode to 14.3.0, to keep this problem in Check. While working on this, I figured that, with the monorepo, Ruby tests stopped working because they were in the wrong folder: I moved them in the right one. ## Changelog: [iOS][Fixed] - Make React Native build with Xcode 14.3.0 and fix tests Reviewed By: blakef Differential Revision: D44605617 fbshipit-source-id: 3ec1f5b36858ef07d9f713d74eb411a1edcccd45
Summary:
On Thursday the 30th, Apple Released Xcode 14.3.0.
This Version of Xcode enforce some version checks for which React-Codegen, which supported iOS 11 as minimum supported version, could not be build anymore.
This change ensue that React-Codegen is always aligned to the min version supported by React Native.
Plus, it moves CircleCI's Xcode to 14.3.0, to keep this problem in Check.
While working on this, I figured that, with the monorepo, Ruby tests stopped working because they were in the wrong folder: I moved them in the right one.
Changelog:
[iOS][Fixed] - Make React Native build with Xcode 14.3.0 and fix tests
Differential Revision: D44605617