Administrator authored 2 years ago and Pierre-Alain Loizeau committed 2 years ago

With the upcoming FairRoot release the function FillEventHeader of the
FairRootManager is deprecated, so use the function from FairSource directly.
Fixes #2644.

Administrator authored 2 years ago and Florian Uhlig committed 2 years ago

When using FairSoft apr22 the compilation without the fix crashes.

There were problems reported when a system version of GTest exist which
should be solved with this fix.

Omveer Singh authored 2 years ago and Omveer_Singh committed 2 years ago

MQ/monitor/CMakeLists.txt: Boost added as dependency
algo/data/CMakeLists.txt: modified in Modern CMake
algo/test/CMakeLists: modified in Modern CMake
core/data/test/global/CMakeLists.txt: Unnecessary dependencies removed and added required dependencies
core/data/test/psd/CMakeLists.txt: Unnecessary dependencies removed
core/detectors/trd/CMakeLists.txt: TMVA dependency change to ROOT::TMVA

Omveer Singh authored 2 years ago and Omveer_Singh committed 2 years ago

Gtest and GtestMain Target created which depend on GTEST target

Remaining files which haven't been changed in modern cmake.
 MQ/monitor : Transition in Modern CMake
 core/data/test: Transition to Modern CMake
 reco/qa: Transition to Modern CMake
 tutorial/TaskToAlgo: Transition to modern CMake

Makes clear to reader that append is default. Allows
previous overwrite option.

There were rich ring finders in the L1 code base which were compiled and linked
into libL1. With the change in the build system this results in a circular
dependency between the L1 and CbmRichReco libraries which was resolved by
removing the classes from compilation.
With this commit the classes are moved from the L1 code base into the rich
reconstruction code base.

Administrator authored 2 years ago and Florian Uhlig committed 2 years ago

Use a different installation directory For the test of the installation to
allow to remove also the build directory.

Test installation also on Debian 10

There was a problem when running CbmRoot from the installation directory with
Debian10. So add also the installation test for Debian10.

On some systems the library seems to be needed in the installation.
See !1005.

part of the same geometry setup

Administrator authored 2 years ago and Florian Uhlig committed 2 years ago

With the change to the CMake targets the dependency tree was broken. The
generation of the ROOT dictionary is now a target on its own which did not
depend on anything except the header files needed for the specific library.
Some of these header files include other header files from the external
libraries. The problem was that now it became possible that the dictionary
generation started before the external header files were available.

Add explicit dependencies from these dictionaries to the external libraries.

Administrator authored 2 years ago and Florian Uhlig committed 2 years ago

OpenMP comes with the proper targets which are used now.