Skip to content
Snippets Groups Projects

Fix use of CLHEP to be compatible with Gaudi new cmake configuration

Merged Sebastien Ponce requested to merge sponce_doubledep into master

When using Gaudi with new CMake configuration CLHEP has to be used in a slightly different way.

This was breaking compilation with DD4hep. The problem has not been completely understood but is due to a mix of 2 ways of finding CLHEP in cmake : findCLHEP.cmake and CLHEPCondig.cmake. We now ensure that only CLHEPCondig.cmake is used.

Edited by Rosen Matev

Merge request reports

Pipeline #1922247 passed

Pipeline passed for 6c6bfbd8 on sponce_doubledep

Approval is optional

Merged by Roel AaijRoel Aaij 4 years ago (Sep 11, 2020 10:39am UTC)

Pipeline #1929175 passed

Pipeline passed for 2f7cb9af 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
  • added bug fix label

  • added Build label

  • Marco Clemencic resolved all threads

    resolved all threads

  • assigned to @frodrigu and unassigned @sponce

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading