Skip to content

Update boost to 1.83.0 - #39552

Closed
Kudo wants to merge 1 commit into
react:mainfrom
Kudo:kudo/boost-183
Closed

Update boost to 1.83.0#39552
Kudo wants to merge 1 commit into
react:mainfrom
Kudo:kudo/boost-183

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, will create another pr to remove that.

Changelog:

[GENERAL][CHANGED] - Update boost to 1.83.0

Test Plan:

  • ci passed
  • build and launch rn-tester 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 labels Sep 20, 2023
@analysis-bot

Copy link
Copy Markdown
PlatformEngineArchSize (bytes)Diff
androidhermesarm64-v8a8,333,777-128
androidhermesarmeabi-v7an/a--
androidhermesx86n/a--
androidhermesx86_64n/a--
androidjscarm64-v8a9,573,191-150
androidjscarmeabi-v7an/a--
androidjscx86n/a--
androidjscx86_64n/a--

Base commit: 7888338
Branch: main

@Kudo
Kudo marked this pull request as ready for review September 20, 2023 10:12
@KudoKudo mentioned this pull request Sep 20, 2023
@Kudo

Kudo commented Sep 20, 2023

Copy link
Copy Markdown
ContributorAuthor

close in favor of #39555

@KudoKudo closed this Sep 20, 2023
@Kudo
Kudo deleted the kudo/boost-183 branch September 20, 2023 10:59
@facebook-github-botfacebook-github-bot added the Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. label Sep 20, 2023
facebook-github-bot pushed a commit that referenced this pull request Sep 20, 2023
Summary:
Remove Xcode 15 `_LIBCPP_ENABLE_CXX17_REMOVED_UNARY_BINARY_FUNCTION` workaround for boost. If we update boost, we don't need the workaround anymore.
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.
## Changelog:
[IOS][CHANGED] - Remove Xcode 15 `_LIBCPP_ENABLE_CXX17_REMOVED_UNARY_BINARY_FUNCTION` workaround for boost
Pull Request resolved: #39555
Test Plan: RNTester + Xcode 15
Reviewed By: cortinico
Differential Revision: D49454494
Pulled By: cipolleschi
fbshipit-source-id: aacabaa130b61598cf8bd206fdbb96d3b50ba74f
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.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.

3 participants

@Kudo@analysis-bot@facebook-github-bot