Skip to content

Conversation

@kraenhansen
Copy link
Member

What, How & Why?

Merging this PR will:

  • Fix an issue introduced by RJS-2101: "Build optimizations" for iOS and Android #6650, where difference in C++ standard used would cause originating from difference in semantics. Suchs as const static variables being initialized into read-only memory at build time vs being initialized at runtime (causing crashes from writing to read-only memory).

☑️ ToDos

  • 📝 Changelog entry
  • 📝 Compatibility label is updated or copied from previous entry
  • 📝 Update COMPATIBILITY.md
  • 🚦 Tests (manually, locally)
  • 📦 Updated internal package version in consuming package.jsons (if updating internal packages)
  • 📱 Check the React Native/other sample apps work if necessary
  • 💥 Breaking label has been applied or is not necessary
  • 🔔 Mention @realm/devdocs if documentation changes are needed

@kraenhansen kraenhansen self-assigned this Aug 10, 2025
@kraenhansen kraenhansen added T-Bug-Crash no-jira-ticket Skip checking the PR title for Jira reference labels Aug 10, 2025
@cla-bot cla-bot bot added the cla: yes label Aug 10, 2025
@kraenhansen kraenhansen force-pushed the kh/fix-prebuild-cxx-standard branch from f4a46a7 to db26375 Compare August 10, 2025 13:14
@kraenhansen kraenhansen marked this pull request as ready for review August 10, 2025 13:16
@kraenhansen kraenhansen merged commit 74289cb into main Aug 10, 2025
58 of 59 checks passed
@kraenhansen kraenhansen deleted the kh/fix-prebuild-cxx-standard branch August 10, 2025 15:08
kraenhansen added a commit that referenced this pull request Aug 11, 2025
* Set C++ standard when prebuilding

* Add note to changelog

* Upgrade wireit
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes no-jira-ticket Skip checking the PR title for Jira reference T-Bug-Crash

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants