Skip to content

Conversation

@cipolleschi
Copy link
Contributor

@cipolleschi cipolleschi commented Apr 1, 2025

Summary:

Xcode 16.3 removes the base template for std::char_traits: developer.apple.com/documentation/xcode-release-notes/xcode-16_3-release-notes

Changelog:

[iOS][Changed] - Bumped folly to fix Xcode 16.3

Test Plan:

Tested that RNTester builds for both iOS and Android with this PR.

@cipolleschi cipolleschi changed the base branch from main to 0.76-stable April 1, 2025 15:08
@facebook-github-bot facebook-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. p: Facebook Partner: Facebook Partner labels Apr 1, 2025
@cipolleschi cipolleschi changed the title cipolleschi/bump folly for xcode163 [RN][iOS][0.76] Bump folly to fix xcode 16.3 removed traits Apr 1, 2025
@cipolleschi cipolleschi marked this pull request as ready for review April 1, 2025 15:09
Copy link
Contributor

@cortinico cortinico left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Comment on lines 10 to -18
spec.name = "fmt"
spec.version = "9.1.0"
spec.version = "11.0.2"
spec.license = { :type => "MIT" }
spec.homepage = "https://github.com/fmtlib/fmt"
spec.summary = "{fmt} is an open-source formatting library for C++. It can be used as a safe and fast alternative to (s)printf and iostreams."
spec.authors = "The fmt contributors"
spec.source = {
:git => fmt_git_url,
:tag => "9.1.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

@cipolleschi Are we bumping FMT also? If so this needs to be mentioned in changelog

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, it is required by folly.

fast_float_git_url = fast_float_config[:git]

Pod::Spec.new do |spec|
spec.name = "fast_float"
Copy link
Contributor

Choose a reason for hiding this comment

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

also this needs to be called out in the changelog

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍

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. p: Facebook Partner: Facebook Partner Pick Request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants