AthToolSupport: support Gaudi interface base classes
Add support for the usage of the Gaudi extends
formalism for interface
declarations in dual-use code:
- Introduce the new
AsgTools/Interfaces.h
header that contains the standalone versions of these classes. Also move theDeclareInterfaceID
here. - This removes the need to use the explicit
declareServiceInterface
calls.
The first commit is the substantial change. The remaining commits are just migrating clients. This change will obviously break any code out there that is still using declareServiceInterface
.
cc @krumnack
Merge request reports
Activity
added full-unit-tests label
mentioned in merge request !74842 (closed)
This merge request affects 9 packages:
- Control/AthOnnx/AthOnnxComps
- Control/AthToolSupport/AsgExampleTools
- Control/AthToolSupport/AsgServices
- Control/AthToolSupport/AsgTools
- InnerDetector/InDetValidation/InDetTrackPerfMon
- PhysicsAnalysis/Algorithms/SelectionHelpers
- PhysicsAnalysis/Algorithms/SystematicsHandles
- PhysicsAnalysis/JetTagging/FlavorTagDiscriminants
- Reconstruction/egamma/egammaMVACalib
Affected files list will not be printed in this case
Adding @robouque ,@cvarni ,@goetz ,@krumnack ,@ligang ,@sroe ,@maparo ,@ssnyder ,@tadej as watchers
added analysis-interface-change label
added analysis-review-expert label
added analysis-review-approved label and removed analysis-review-expert analysis-review-required labels
CI Result SUCCESS (hash d75d06cc)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis 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
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output (remote access info)removed review-pending-level-1 label
added review-approved label
mentioned in commit 7fa578ef