Skip to content
Snippets Groups Projects

Fix build system for Apple Clang 17

Merged Florian Uhlig requested to merge f.uhlig/cbmroot:fix_macosx_clang17 into master
All threads resolved!

Fix compilation errors for Apple Arm

The flag -msse is only a valid one for the x86 platform. Up to now only
a warning was shown if the flag was used on the arm platform. With
Apple Clang 17 using the flag is now an error.
Remove the flag when building for arm.

Fix compilation error due to a missing template argument in external
package.

Allow compilation with specific SDK an macosx

With the latest version of macosx and/or Clang it is not possible to
compile older ROOT versiond (v6.30.08) with the default Software
Development Kit (SDK). To compile with a non default one it is needed
to pass the extra paramter -DCMAKE_OSX_SYSROOT when running CMake.
The flag needs to be added when compiling FairSoft, FairRoot and
CbmRoot.

This commit properly pass this flag also to all our external packages
such that they are also properly build.
Also some cleanup of the build system files for the external packages
is done. Some default CMake parameters are defined only once and then
used in any of the recipes which are used to build the external
packages.

Merge request reports

Merge request pipeline #35278 passed

Merge request pipeline passed for 493a9a08

Merged by Pierre-Alain LoizeauPierre-Alain Loizeau 1 month ago (Jul 8, 2025 11:29am UTC)

Loading

Pipeline #35280 passed

Pipeline passed for 493a9a08 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Looks fine to me as far as I understand the changes. I will test on my system, but currently have still problems installing FairRoot (see #3654).

  • Florian Uhlig added 18 commits

    added 18 commits

    Compare with previous version

  • Florian Uhlig added 9 commits

    added 9 commits

    Compare with previous version

  • Florian Uhlig resolved all threads

    resolved all threads

  • Volker Friese approved this merge request

    approved this merge request

  • Volker Friese added 10 commits

    added 10 commits

    Compare with previous version

  • Dirk Hutter mentioned in merge request !2166 (merged)

    mentioned in merge request !2166 (merged)

  • Please register or sign in to reply
    Loading