Skip to content

Update boost to 1.83.0 - #39555

Closed
Kudo wants to merge 2 commits into
react:mainfrom
Kudo:kudo/remove-xcode-boost-workaround
Closed

Update boost to 1.83.0#39555
Kudo wants to merge 2 commits into
react:mainfrom
Kudo:kudo/remove-xcode-boost-workaround

Conversation

@Kudo

@KudoKudo commented Sep 20, 2023

Copy link
Copy Markdown
Contributor

Summary:

Since folly is updated from 17154a6, it is good to bump boost too.

After updating boost, we can remove the Xcode 15 _LIBCPP_ENABLE_CXX17_REMOVED_UNARY_BINARY_FUNCTION workaround which originally to fix Xcode 15 for boost.

Changelog:

[GENERAL][CHANGED] - Update boost to 1.83.0

Test Plan:

  • CI passed
  • Build and launch RNTester on Android and iOS

@facebook-github-botfacebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Contributor A React Native contributor. p: Expo Partner: Expo Partner Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. labels Sep 20, 2023

@cipolleschicipolleschi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @Kudo for taking care of this! 🙌

@cipolleschi

Copy link
Copy Markdown
Contributor

This PR is based on #39552. Not sure how Meta internally deal with stacked PRs. This PR just adds a new commit based on #39552 anyway.

I think that this PR contains all the changes that are in #39552, so we can land this one only. Could you double-check that as well?

@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

@Kudo

Kudo commented Sep 20, 2023

Copy link
Copy Markdown
ContributorAuthor

I think that this PR contains all the changes that are in #39552, so we can land this one only. Could you double-check that as well?

that's correct! good to know we can just land this one. let me update this pr's description/changelog and close #39552.

@KudoKudo changed the title Remove xcode 15 workaround for boostUpdate boost to 1.83.0Sep 20, 2023
@KudoKudo mentioned this pull request Sep 20, 2023
@analysis-bot

Copy link
Copy Markdown
PlatformEngineArchSize (bytes)Diff
androidhermesarm64-v8a8,333,695-210
androidhermesarmeabi-v7an/a--
androidhermesx86n/a--
androidhermesx86_64n/a--
androidjscarm64-v8a9,573,167-174
androidjscarmeabi-v7an/a--
androidjscx86n/a--
androidjscx86_64n/a--

Base commit: 7888338
Branch: main

@facebook-github-botfacebook-github-bot added the Merged This PR has been merged. label Sep 20, 2023
@facebook-github-bot

Copy link
Copy Markdown
Contributor

@cipolleschi merged this pull request in b9f0bdd.

@liamjones

Copy link
Copy Markdown
Contributor

Something I've just noticed in relation to this PR when upgrading from 0.72.x to 0.73.x - if the _LIBCPP_ENABLE_CXX17_REMOVED_UNARY_BINARY_FUNCTION directive had been added to a project previously, it doesn't get removed again here.

Is that an issue or not? I noticed it's a CXX17 directive and 0.73 moves to c++20 so maybe it isn't obeyed anyway?

@cipolleschi

Copy link
Copy Markdown
Contributor

@liamjones it should not be an issue. The directive is needed to backport the name of an API that has changed. I don't think that it's an issue if we keep it. Unless you are seeing some specific errors: if that's the case, could you share them?

@liamjones

liamjones commented May 21, 2024

Copy link
Copy Markdown
Contributor

@cipolleschi Nope, not seeing any errors. I just noticed it wasn't being removed when I did the RN upgrade and wanted to check if it was deliberate. Thanks for confirming it was! 😄

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.ContributorA React Native contributor.MergedThis PR has been merged.p: ExpoPartner: ExpoPartnerShared with MetaApplied via automation to indicate that an Issue or Pull Request has been shared with the team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@Kudo@cipolleschi@facebook-github-bot@analysis-bot@liamjones