PanTauAlgs Build Cleanup, master branch (2021.03.22.)
The main motivation was to make sure that the public headers of the package would be properly installed in "Athena mode", in order for the dictionary to work correctly.
But the cleanup grew into something more. I replaced the CINT dictionary of the package into a Reflex one, introduced the PanTauAlgsLib
library that would be built in all circumstances, cleaned up the dependencies of the package, and made sure that the dictionary would refer to types with their correct names.
This is all primarily meant to fix the issue described in ATEAM-721. To get rid of the following types of messages with the upcoming ROOT version:
...
15:08:30 Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
15:08:30 Missing FileEntry for PanTauAlgs/ITool_PanTauTools.h
15:08:30 requested to autoload type PanTau::ITool_PanTauTools
15:08:30 Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
15:08:30 Missing FileEntry for PanTauAlgs/ITool_InformationStore.h
15:08:30 requested to autoload type PanTau::ITool_InformationStore
15:08:30 Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
15:08:30 Missing FileEntry for PanTauAlgs/ITool_InputConverter.h
15:08:30 requested to autoload type PanTau::ITool_InputConverter
15:08:30 Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
15:08:30 Missing FileEntry for PanTauAlgs/ITool_ModeDiscriminator.h
15:08:30 requested to autoload type PanTau::ITool_ModeDiscriminator
15:08:30 Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
15:08:30 Missing FileEntry for PanTauAlgs/ITool_TauConstituentGetter.h
15:08:30 requested to autoload type PanTau::ITool_TauConstituentGetter
15:08:30 Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
15:08:30 Missing FileEntry for PanTauAlgs/ITool_TauConstituentSelector.h
15:08:30 requested to autoload type PanTau::ITool_TauConstituentSelector
15:08:30 Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
15:08:30 Missing FileEntry for PanTauAlgs/ITool_DetailsArranger.h
15:08:30 requested to autoload type PanTau::ITool_DetailsArranger
15:08:30 Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
15:08:30 Missing FileEntry for PanTauAlgs/Tool_DecayModeDeterminator.h
15:08:30 requested to autoload type PanTau::Tool_DecayModeDeterminator
15:08:30 Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
15:08:30 Missing FileEntry for PanTauAlgs/Tool_DetailsArranger.h
15:08:30 requested to autoload type PanTau::Tool_DetailsArranger
15:08:30 Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
15:08:30 Missing FileEntry for PanTauAlgs/Tool_FeatureExtractor.h
15:08:30 requested to autoload type PanTau::Tool_FeatureExtractor
15:08:30 Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
15:08:30 Missing FileEntry for PanTauAlgs/Tool_InformationStore.h
15:08:30 requested to autoload type PanTau::Tool_InformationStore
15:08:30 Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
15:08:30 Missing FileEntry for PanTauAlgs/Tool_InputConverter.h
15:08:30 requested to autoload type PanTau::Tool_InputConverter
15:08:30 Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
15:08:30 Missing FileEntry for PanTauAlgs/Tool_ModeDiscriminator.h
15:08:30 requested to autoload type PanTau::Tool_ModeDiscriminator
15:08:30 Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
15:08:30 Missing FileEntry for PanTauAlgs/Tool_TauConstituentGetter.h
15:08:30 requested to autoload type PanTau::Tool_TauConstituentGetter
15:08:30 Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
15:08:30 Missing FileEntry for PanTauAlgs/Tool_TauConstituentSelector.h
15:08:30 requested to autoload type PanTau::Tool_TauConstituentSelector
15:08:30 TauRecRunnerAlg.tauRec_PanTau 0 INFO TauRecRunnerAlg.tauRec_PanTau initialize()
15:08:30 ToolSvc.PanTau_InformationStore 0 INFO ToolSvc.PanTau_InformationStore initialize()
...
Unfortunately the latest master--dev4LCG
nightlies are not in the greatest shape, so I didn't test whether these changes actually fix this issue.
Note that clients of the package in AnalysisBase
will have to switch from linking against PanTauAlgs
to using PanTauAlgsLib
from now on. (There are no clients in this repository, only in "user repositories".) But this is as good a time as any to force them into making that change...
Merge request reports
Activity
added bugfix label
- Resolved by Attila Krasznahorkay
This merge request affects 1 package:
- Reconstruction/PanTau/PanTauAlgs
This merge request affects 5 files:
- Reconstruction/PanTau/PanTauAlgs/CMakeLists.txt
- Reconstruction/PanTau/PanTauAlgs/PanTauAlgs/PanTauAlgsDict.h
- Reconstruction/PanTau/PanTauAlgs/PanTauAlgs/ToolsDef.h
- Reconstruction/PanTau/PanTauAlgs/PanTauAlgs/selection.xml
- Reconstruction/PanTau/PanTauAlgs/Root/LinkDef.h
added Reconstruction Tau master review-pending-level-1 labels
- Resolved by Alaettin Serhan Mete
When testing these changes on top of
Athena,master--dev4LCG,r19
(instead ofAthena,master--dev4LCG,latest
) I can confirm that the ROOT error messages are gone. Good...
mentioned in merge request !41827 (merged)
CI Result SUCCESS (hash 1ff11d26)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon externals cmake make required tests optional tests Full details available on this CI monitor view
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
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 30375]added review-approved label and removed review-pending-level-1 label
mentioned in commit c11a6bb2
added sweep:ignore label