Skip to content

Conversation

kvakvs
Copy link
Collaborator

@kvakvs kvakvs commented Aug 7, 2025

Fixes #9972
The PR disables one compiler hardening option when compiling wx C++ wrapper code, and only on MacOS (darwin).

The hardening options were set to on by default in OTP 28 and having them enabled breaks C++ exception propagation, making exceptions fall through into the C abort function which ends the VM.

@kvakvs kvakvs requested a review from dgud August 7, 2025 15:22
@kvakvs kvakvs self-assigned this Aug 7, 2025
@kvakvs kvakvs added team:PS Assigned to OTP team PS testing currently being tested, tag is used by OTP internal CI labels Aug 7, 2025
Copy link
Contributor

github-actions bot commented Aug 7, 2025

CT Test Results

  2 files   17 suites   4m 21s ⏱️
157 tests 150 ✅ 7 💤 0 ❌
173 runs  166 ✅ 7 💤 0 ❌

Results for commit 5510216.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

@kvakvs kvakvs force-pushed the eimytdl/otp28-wx-crashes/OTP-19724 branch from baa5be4 to a54264d Compare August 7, 2025 15:40
@kvakvs kvakvs removed the testing currently being tested, tag is used by OTP internal CI label Aug 20, 2025
@kvakvs kvakvs force-pushed the eimytdl/otp28-wx-crashes/OTP-19724 branch from a54264d to 54fcf6b Compare August 25, 2025 16:24
@kvakvs kvakvs force-pushed the eimytdl/otp28-wx-crashes/OTP-19724 branch from 54fcf6b to 5510216 Compare August 25, 2025 16:26
@kvakvs kvakvs added the testing currently being tested, tag is used by OTP internal CI label Aug 27, 2025
@IngelaAndin IngelaAndin requested a review from dgud September 2, 2025 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team:PS Assigned to OTP team PS testing currently being tested, tag is used by OTP internal CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants