Move analysis packages to Analysis. Merge DaVinciUtils into DaVinciKernel

Removing the following packages:

• Phys/DaVinciDecayFinder/
• Phys/DaVinciTools
• Phys/DSTWriters
• Phys/DaVinciUtils

The first three are moved to Analysis in Analysis!751 (merged)

The three headers in Phys/DaVinciUtils are only used in Phys. I thus move them to DaVinciKernel and adapt the include statements of the clients.

Goes together with Analysis!751 (merged)

added cleanup lhcb-run3-cleanup labels

marked this merge request as draft

changed the description

requested review from @rmatev

mentioned in issue #19 (closed)

• [2021-02-09 01:02] Validation started with lhcb-run3-cleanup#131
• [2021-02-10 00:38] Validation started with lhcb-run3-cleanup#132
• [2021-02-11 01:23] Validation started with lhcb-run3-cleanup#133
• [2021-02-12 01:21] Validation started with lhcb-run3-cleanup#134
• [2021-02-13 01:06] Validation started with lhcb-run3-cleanup#135
• [2021-02-14 00:36] Validation started with lhcb-run3-cleanup#136
• [2021-02-15 00:51] Validation started with lhcb-run3-cleanup#137
• [2021-02-16 01:16] Validation started with lhcb-run3-cleanup#138
• [2021-02-17 01:02] Validation started with lhcb-run3-cleanup#139
• [2021-02-17 15:12] Validation started with lhcb-master-mr#1889

This didn't build this night. Seems DaVinciKernel needs a header from DaVinciUtils. I'll investigate.

mentioned in merge request Analysis!751 (merged)

added 1 commit

• 58327ca4 - Moved DaVinciUtils headers to DaVinciKernel

Compare with previous version

changed title from Draft: Moving analysis pacakges to Analysis to Draft: Move analysis pacakges to Analysis. Merge DaVinciUtils into DaVinciKernel

changed the description

added 1 commit

• 5b0772ca - Fixed formatting

Compare with previous version

added 1 commit

• 079b183c - Remove spurious dependency on DaVinciTools

Compare with previous version

changed title from Draft: Move analysis pacakges to Analysis. Merge DaVinciUtils into DaVinciKernel to Draft: Move analysis packages to Analysis. Merge DaVinciUtils into DaVinciKernel

assigned to @frodrigu and unassigned @lmeyerga

marked this merge request as ready

approved this merge request

All good for me.

Suggest to add the all-slots label here, so it goes with Analysis!751 (merged)