Skip to content

[MOO-2251]: App crash fix for debug build due to folly#1179

Open
saurabhchavan1711 wants to merge 1 commit intorelease/17.x.xfrom
moo/MOO-2251-Appcrash-fix-due-to-folly-for-release-17
Open

[MOO-2251]: App crash fix for debug build due to folly#1179
saurabhchavan1711 wants to merge 1 commit intorelease/17.x.xfrom
moo/MOO-2251-Appcrash-fix-due-to-folly-for-release-17

Conversation

@saurabhchavan1711
Copy link

@saurabhchavan1711 saurabhchavan1711 commented Mar 23, 2026

Description

Problem
Debug builds crash with SIGABRT in F14SetFallback.h:234 on Xcode 26 when the New Architecture passes data through F14 hash maps.

Root Cause
Folly uses reinterpret_cast to access std::unordered_set private internals — an unsafe hack. Xcode 26's libc++ changed the internal layout, causing a debug-only assertion (FOLLY_SAFE_DCHECK) to fail. Release builds are unaffected as they already define NDEBUG.

Fix
Define NDEBUG=1 for all pod targets in Debug configuration, matching Release behavior. Must apply to all targets because F14SetFallback.h is a header-only template compiled by any including target.

Checklist

To ensure this pull request meets the requirements for merging, please complete the checklist below:

  • Release Note: As a part of the release process, an automated PR will be created for the Mendix Docs repository relevant to the changes introduced in this PR. Post release, please ensure that the release note accurately reflects the changes and impacts of this PR.
  • Breaking Changes: This PR introduces breaking changes (e.g., changes that require updates to existing configurations, dependencies).
    • If yes, I have documented these breaking changes and provided guidance for users to adapt.
    • Details about breaking changes: [Provide details here, if applicable]

This PR contains

  • Bug fix
  • Feature
  • Refactor
  • Documentation
  • Other (describe)

Important Notes

  • Make sure the release note accurately describes the changes and impacts introduced by this PR.

@github-actions github-actions bot added the missing-docs-pr Indicates that the current PR does not reference a corresponding documentation PR label Mar 23, 2026
@github-actions
Copy link
Contributor

Hi @saurabhchavan1711, please add a link to the corresponding docs PR in the description. If documentation update is not required for this PR, you can add a comment with 'skip-docs-check' to skip this check.

@github-actions github-actions bot removed the missing-docs-pr Indicates that the current PR does not reference a corresponding documentation PR label Mar 23, 2026
@saurabhchavan1711 saurabhchavan1711 changed the title app crash fix for debug build due to folly [MOO-2251]: App crash fix for debug build due to folly Mar 23, 2026
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.

1 participant