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

Add support for building with Xcode 15 #37758

Closed
wants to merge 2 commits into from
Closed

Add support for building with Xcode 15 #37758

wants to merge 2 commits into from

Conversation

AlexanderEggers
Copy link
Contributor

Summary:

Fixes #37748

This PR adds a patch which fixes a build issue in Xcode 15.

Changelog:

[IOS] [ADDED] - Add support for building with Xcode 15

Test Plan:

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jun 7, 2023
Copy link
Contributor

@cipolleschi cipolleschi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems great to me, thank you for taking the time to do this and to also write tests! :D

.each do |pod_name, target_installation_result|
target_installation_result.native_target.build_configurations.each do |config|
# unary_function and binary_function are no longer provided in C++17 and newer standard modes as part of Xcode 15.
# They can be re-enabled with setting _LIBCPP_ENABLE_CXX17_REMOVED_UNARY_BINARY_FUNCTION
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: could you add a link to the Xcode doc about this point?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

@AlexanderEggers AlexanderEggers Jun 7, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cipolleschi I have added the link to that part of the code. Let me know if you prefer pointing to a different URL.

@facebook-github-bot
Copy link
Contributor

@cipolleschi has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@analysis-bot
Copy link

analysis-bot commented Jun 7, 2023

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 8,744,421 -3
android hermes armeabi-v7a 8,056,625 -6
android hermes x86 9,235,547 -5
android hermes x86_64 9,086,380 -1
android jsc arm64-v8a 9,307,265 -1
android jsc armeabi-v7a 8,496,805 -8
android jsc x86 9,369,282 -2
android jsc x86_64 9,624,203 -2

Base commit: e449706
Branch: main

@facebook-github-bot
Copy link
Contributor

@cipolleschi has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Jun 8, 2023
@facebook-github-bot
Copy link
Contributor

@cipolleschi merged this pull request in 10d5588.

@AlexanderEggers AlexanderEggers deleted the feature/add-xcode15-support branch June 8, 2023 11:32
kelset pushed a commit that referenced this pull request Jun 13, 2023
Summary:
Fixes #37748

This PR adds a patch which fixes a build issue in Xcode 15.

## Changelog:

[IOS] [ADDED] - Add support for building with Xcode 15

Pull Request resolved: #37758

Reviewed By: cortinico

Differential Revision: D46524009

Pulled By: cipolleschi

fbshipit-source-id: 9f6c12e12a15c154467282a7b4a00e80e5cc0af2
Kudo pushed a commit to expo/react-native that referenced this pull request Jun 15, 2023
Summary:
Fixes facebook#37748

This PR adds a patch which fixes a build issue in Xcode 15.

## Changelog:

[IOS] [ADDED] - Add support for building with Xcode 15

Pull Request resolved: facebook#37758

Reviewed By: cortinico

Differential Revision: D46524009

Pulled By: cipolleschi

fbshipit-source-id: 9f6c12e12a15c154467282a7b4a00e80e5cc0af2
(cherry picked from commit 8ed2cfd)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Build fails for Xcode 15 Beta 1 (and Beta 7)
4 participants