Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • athena athena
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Jira
    • Jira
  • Merge requests 179
    • Merge requests 179
  • Deployments
    • Deployments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • Code review
    • Issue
    • Repository
  • Activity
  • Graph
  • Commits
Collapse sidebar
  • atlas
  • athenaathena
  • Merge requests
  • !48699

Merged
Created Dec 01, 2021 by Attila Krasznahorkay@akrasznaMaintainer

Analysis HepMC CMake Update, master branch (2021.12.01.)

  • Overview 19
  • Commits 2
  • Pipelines 2
  • Changes 2

Now that HepMC (or hepmc3) is used through AtlasHepMCLib, TruthUtils and MCTruthClassifier should no longer make decisions based on whether HepMC is available or not. As it would have to replicate the logic of AtlasHepMC to do this correctly.

Instead, the decision whether or not to link against AtlasHepMCLib is now made purely based on XAOD_STANDALONE. Since it's only AnalysisBase that does not come with AtlasHepMC.

At the same time removed the optional usage of HEPUtils and MCUtils in TruthUtils. Since both of those are always available, in all of the projects that we build. So it's an error if either of them is not found.

This is all to fix ATLASG-1616.

P.S. The usage of TruthUtilsEnvironmentConfig.cmake should be made a little nicer as well. But that will need some update in the AtlasLCG code, so let's leave that for a future MR. 😉

Edited Dec 02, 2021 by Attila Krasznahorkay
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: TruthUtilsWarningFix-master-20211201