Skip to content

macOS build: AppleClang 21 fix + workflow artifact sync + Lima README#26

Draft
TomKalina wants to merge 2 commits into
FULU-Foundation:mainfrom
TomKalina:fix/macos-build
Draft

macOS build: AppleClang 21 fix + workflow artifact sync + Lima README#26
TomKalina wants to merge 2 commits into
FULU-Foundation:mainfrom
TomKalina:fix/macos-build

Conversation

@TomKalina

Copy link
Copy Markdown

Summary

  • Fix wxMediaCtrl2.h constexpr error on AppleClang 21 (Xcode 26+) — out-of-enum-range cast no longer a constant expression in C++20; mirror the static inline const + static_cast pattern already used in MediaPlayCtrl.h:74
  • Sync .github/workflows/build_{macos,windows}_bridge.yml with the de-collided artifact names already staged at repo root (linux_host_runtime_macos_* / _windows_*) so my_build_all.yml doesn't fail on artifact-name collision when macOS + Windows run in parallel
  • README: document Lima + Rosetta prerequisites for the macOS bridge runtime

Test plan

  • Local arm64 deps build (build_release_macos.sh -d -x -a arm64 -t 11.3)
  • Local arm64 slicer build (after the wxMediaCtrl2 fix)
  • CLI smoke test (OrcaSlicer --help, exits 0)
  • GUI launch smoke test (Setup Wizard + main window open)
  • CI build_macos_bridge.yml green end-to-end (Linux bridge job + arm64 + x86_64 + universal DMG)
  • CI build_windows_bridge.yml still green after artifact rename
  • Verify Bambu cloud login works once Lima VM is bootstrapped on a clean Mac

TomKalina added 2 commits May 15, 2026 10:08
- README: add Lima/Rosetta prerequisites for macOS bridge runtime
- workflows: sync .github/workflows/{build_macos,build_windows}_bridge.yml with the de-collided artifact names already staged at repo root (linux_host_runtime_macos_* / _windows_*)
AppleClang 21 (Xcode 26+) enforces the C++20 rule that out-of-enum-range
casts are not constant expressions. wxMediaState's max value is 2, so
(wxMediaState)6 is no longer constexpr.

Mirrors the static inline const + static_cast pattern already used in
MediaPlayCtrl.h:74.
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