Revise the build environment (Move to C++17)
Use C++17 as C++ standard for the compilation of CbmRoot. Check if compiler supports the C++17 standard. Check if also FairSoft was compiled with C++17. Check the required versions of ROOT (at least Root6), FairSoft( at least apr21) and FairRoot (at leass v18.6.7). If any of the checks fail stop the configuration with a meaningful error message. Remove some switches which are not needed any longer. Remove checks for C++11 features. No need to check them any longer, since the new standard is noe C++17.
Merge request reports
Activity
added Build System CI/CD labels
requested review from @p.-a.loizeau
assigned to @v.friese
added 7 commits
Toggle commit listadded 1 commit
- 90f3c337 - Add missing dependency for clang-format checks
added 2 commits
Dear @f.uhlig, @v.friese, @p.-a.loizeau, @s.lebedev,
you have been identified as code owner of at least one file which was changed with this merge request.
Please check the changes and approve them or request changes.
added CodeOwners label
I think the MR is ready to be merged but I am not sure if now is a good time for the change. Many people are currently working at the CBM@SIS18 proposal. Since the MR should not interfere with any other changes it should be possible to merge it any time. To avoid any accidental merge I put the MR in Draft mode. Feel free to change that whenever you want to merge the MR.
to get the real data test running I had to install the latest FairSoft and FairRoot versions on run4. I installed them alongside the older versions in /opt/cbmsoft so I don't expect any problems.
Edited by Florian Uhlig- Resolved by Pierre-Alain Loizeau
mentioned in merge request !792 (merged)