Also move other interface includes to main header file.

Remove `atlas_depends_on_subdirs`, update link dependencies and do not
install (private) component headers.

    
Remove reference to obsolete header StoreGate.h.

The single RegSelSvc is to be replaced by a separate RegSelTool for each subsystem.

This merge request is to implement that migration for the ID code.

This code contains about 60 % of all the changes that are needed for the migration,
and most of the changes for the pixel, SCT and TRT.

```
For information the packages needing changes are
Innerdetector/InDetEventCnv/PixelRawDataByteStreamCnv
InnerDetector/InDetEventCnv/TRT_RawDataByteStreamCnv
InnerDetector/InDetExample/InDetRecExample
InnerDetector/InDetRecAlgs/InDetPrepRawDataFormation
InnerDetector/InDetRecTools/TRT_SeededSpacePointFinderTool

Reconstruction/TRT_Rec
Reconstruction/iPat/iPatRecExample

Innerdetector/InDetExample/InDetTrigRecExample

InnerDetector/InDetTrigRecAlgs/InDetTrigPrepRawDataFormat
InnerDetector/InDetTrigRecAlgs/InDetTrigRawDataProvider
InnerDetector/InDetTrigRecAlgs/SiTrigSPSeededTrackFinder
InnerDetector/InDetTrigRecAlgs/SiTrigSpacePointFormation
InnerDetector/InDetTrigRecAlgs/TRT_TrigSeededTrackFinder
InnerDetector/InDetTrigRecAlgs/TRT_TrigTrackSegmentsFinder

Trigger/TrigTools/TrigOnlineSpacePointTool
Trigger/TriggerCommon/TriggerJobOpts

Trigger/TrigAlgorithms/TrigGenericAlgs
Trigger/TrigAlgorithms/TrigT2MinBias
Trigger/TrigHypothesis/TrigLongLivedParticlesHypo
```

plus the configuratin for every package that uses any of the above packages which will need to
instantiate the RegSelTools for the tool and algorithms configurables.

Susumu Oda authored 4 years ago and Adam Edward Barton committed 4 years ago

Use IDCInDetBSErrContainer for SCT_FlaggedCondData instead of SCT_FlaggedCondData in WriteInDetESD.py

Library dependency fix.

Library dependency fixes.

Library dependency fixes.

Library dependency fixes.

Library dependency fixes.

Library dependency fixes.

Library dependency fixes.

Library dependency fixes.

Currently the InDetTrigExtensProcessor relies on the scoring tool to
create the track summary and store it in the track, which violates
const-ness.

Move the reasons of trhead unsafety to the lines of ATLAS_NOT_THREAD_SAFE or ATLAS_NO_CHECK_FILE_THREAD_SAFETY for easier understanding using LXR and git grep. Remove ATLAS_NOT_THREAD_SAFE from TRT_BSErrContainerCnv_p1::transToPers and TRT_BSErrContainerCnv_p2::transToPers.

Walter Lampl authored 4 years ago and Edward Moyse committed 4 years ago

rename TRT_BSErrContainer into TRT_BSErrContainer_p1 (to serve as persistent version), create new transient TRT_BSErrContainer mimicking the interface of the TRT_ByteStream_ConditionsSvc

Change TrigTrackSlimmer to use the slimCopy() method of ITrackSlimmingTool
rather than slim.

Avoids thread-safety issues.

The thread-safety checker was not properly checking calls to virtual functions.
When that is fixed, we get a warning about the call to
IPRDtoTrackMapExchange::setPRDtoTrackMap.

This is indeed not thread safe, and is supposed to be used only for
backwards compatibility with run 2.

To resolve this, split out the non-thread-safe part into a separate function.
We suppress the thread-safety checker warning in the call to it,
but we also explicitly check and fail the job if we're running
such a run 2 configuration with more than one thread.

Susumu Oda authored 4 years ago and Frank Winklmeier committed 4 years ago

Add using std::abs as protection against missing #include <cmath> when  using namespace std is used.