RICH Preparations for DD4Hep support

See description in LHCb!3355 (merged)

added RICH label

mentioned in merge request LHCb!3355 (merged)

approved this merge request

added 3 commits

• ac9caa5c...35a50393 - 2 commits from branch lhcb:master
• e22e837f - MCRichDigitsToRawBufferAlg - Adapt to change in helpers due to removal of...

Compare with previous version

• [2021-12-08 13:27] Validation started with lhcb-master-mr#3290
• [2021-12-08 18:16] Validation started with lhcb-master-mr#3293
• [2021-12-09 19:06] Validation started with lhcb-master-mr#3311
• [2021-12-09 19:21] Validation started with lhcb-master-mr#3311
• [2021-12-09 19:35] Validation started with lhcb-master-mr#3311
• [2021-12-10 12:08] Validation started with lhcb-master-mr#3316

changed title from Preparations for DD4Hep support to RICH Preparations for DD4Hep support

added 9 commits

• e22e837f...e454e79d - 8 commits from branch lhcb:master
• 41ff543c - MCRichDigitsToRawBufferAlg - Adapt to change in helpers due to removal of...

Compare with previous version

assigned to @abrearod

added modernisation label

approved this merge request

merged

mentioned in commit b60c2fa1

mentioned in commit 24280e32

mentioned in merge request !373 (merged)

mentioned in commit 4127c82d

@jonrob could you please fix the clang12 compilation warning introduced by this MR? https://lhcb-nightlies.web.cern.ch/nightly/lhcb-gaudi-head/3118/Boole/x86_64_v2-centos7-clang12-opt/build#Rich/RichReadout_2

@rmatev why was this merged without fixing the warning?

Sorry, ignore, done in !373 (merged) a few minutes ago...

Thanks @rmatev (also for the LHCB fix). Forgot to check the clang12 builds...

mentioned in issue Moore#362 (closed)