Skip to content

Temp fix iOS build for Xcode 26.4#121

Merged
nw-sentry merged 1 commit intomasterfrom
nic/xcode-26-update
Apr 21, 2026
Merged

Temp fix iOS build for Xcode 26.4#121
nw-sentry merged 1 commit intomasterfrom
nic/xcode-26-update

Conversation

@nw-sentry
Copy link
Copy Markdown
Contributor

@nw-sentry nw-sentry commented Apr 20, 2026

Summary

Latest Xcode 26.4's newer clang rejects the fmt pod's FMT_COMPILE_STRING path with
"call to consteval function 'fmt::basic_format_string<...>' is not a constant expression"

This breaks pod installxcodebuild

This PR works around that by building the fmt pod against C++17 where the consteval constant-evaluation is more lenient, unblocking local iOS builds without touching React Native or Sentry SDK versions.

Changes

  • ios/Podfile: Added a post_install block that sets CLANG_CXX_LANGUAGE_STANDARD = c++17 on the fmt pod's build configurations.
  • ios/Podfile.lock: Updated PODFILE CHECKSUM (automatic)
  • ios/sentry_react_native.xcodeproj/project.pbxproj: CocoaPods integration churn written by pod install under CocoaPods 1.16.2 — drops redundant empty inputPaths/outputPaths entries on the [CP] Embed Pods Frameworks and [CP] Copy Pods Resources script phases, and clears a stale CC override that pointed at the ccache-clang wrapper. No behavioral change beyond making the committed project match what CocoaPods generates on current toolchains.

Test plan

  • cd ios && pod install completes cleanly on Xcode 26.4
  • npm run build-dev-ios builds and launches on the iOS simulator
  • Android build (npm run build-dev-android) unaffected

@nw-sentry nw-sentry requested a review from antonis April 20, 2026 21:47
Copy link
Copy Markdown
Collaborator

@antonis antonis left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@nw-sentry nw-sentry merged commit ed40fea into master Apr 21, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants