Acts CMake Fix(es), main branch (2024.02.07.)
These are some fixes for the CMake configuration of the Acts packages. Making sure that libraries would be set up with the correct types and contents, and that libraries would declare their dependencies correctly.
The whole thing was triggered by the bug in the configuration of ActsEventCnv
. (Leading to sources in that package being compiled/linked into all clients linking against ActsEventCnvLib
.) Then I set out to review the configuration of all packages under Tracking/Acts
, but had to give up.
In order to figure out what "things" a given package depends on, I was executing commands like the following inside of the package:
grep -rh "^#include" * | sed -rn 's:#include ["<](.+)/.*:\1:p' | sort | uniq
For packages/libraries where the "public" and "private" dependencies matter, I would execute this command like:
grep -rh "^#include" ActsEvent/* | sed -rn 's:#include ["<](.+)/.*:\1:p' | sort | uniq
grep -rh "^#include" Root/* | sed -rn 's:#include ["<](.+)/.*:\1:p' | sort | uniq
, and then construct the library list from these outputs.
Note that I can't guarantee that my updates don't break any downstream packages/libraries, so let's see what the CI says. (The configurations that I touched are now correct, but in CMake configurations two wrongs can sometimes make a right. So we'll see.)
Merge request reports
Activity
added bugfix label
added ACTS Tracking main review-pending-level-1 labels
CI Result SUCCESS (hash 76e08985)Athena externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 4157]added review-approved label and removed review-pending-level-1 label
mentioned in commit 29c62c65
mentioned in merge request !68689 (merged)