-
Notifications
You must be signed in to change notification settings - Fork 168
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
RCORE-2166 Fix building with Xcode 16 #7802
Conversation
Pull Request Test Coverage Report for Build thomas.goyne_403Details
💛 - Coveralls |
Pull Request Test Coverage Report for Build thomas.goyne_404Details
💛 - Coveralls |
Pull Request Test Coverage Report for Build thomas.goyne_407Details
💛 - Coveralls |
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.
Do build-apple-device.sh
and/or build-cocoa.sh
need updates? build-apple-device.sh
seemed to be pretty lightweight and I don't think it needs any, but not sure about build-cocoa.sh
.
Or are these even used anymore? They used to be used by Jenkins, but not sure if anyone still uses them.
They're used by Cocoa's GHA to build core binaries and nothing else. |
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.
LGTM
All of our supported versions now support always using the iphoneos SDKROOT, so we can simplify that logic and not require updates for each major version.
Cocoa has been building the SDK as C++20 for the last few years, but with Xcode 16 the mix of C++17 and C++20 is actually causing problems due to ODR violations. There's no obvious downside to just building the cocoa-specific binaries as c++20.
Fixes #7801.