Enable thread-safety checking.

Dropped the redundant AbsMaterialManager interface, and removed all references to this
interface from client packages.

Dropped redundant methods from RDBMaterialManager, and in this way got rid of lots of
duplicated code.

- remove `atlas_depends_on_subdirs`
- update link dependencies
- replace `atlas_install_headers` with interface library or remove it
  entirely if headers not used outside package
- enable flake8 where applicable

There is already a library that provides the public headers and hence
the `atlas_install_headers` is redundant.

If a dtor is declared final, the class itself should also be declared final.

If a dtor is declared final, the class itself should also be declared final.

Library dependency fixes.

Library dependency fixes.

This package has a component library, but it also exports headers.
Define an interface library for the headers.

Library dependency fixes.

StatusCodeSvc is deprecated. Remove all references to it from job
options.

In the "new packaging" the external is now called like that...

Updated `ComponentAccumulator` configuration tests which dump the configuration to a pickle file.
In python 3 the output file needs to be explicitly opened in binary mode.
I.e.
```
f = open('configdump.pkl', 'wb')
```
rather than
```
f = open('configdump.pkl', 'w')
```

Fix warnings/errors flagged by `flake8`. Mostly unused variables/imports
or stylistic code problems.

Remove danging uses of CLHEP macros.

Given the standalone library is based on Eigen instead of CLHEP, all GeoModel clients
in Athena code were also migrated to the Eigen-based interface of GeoModel. This affected
a large number of packages. Most significant changes have been introduced to subsystem
GeoModel packages.

The in-house GeoModelKernel code has been removed from the repository.

Almost all header files in PixelGeoModel have been moved to the src directory.
The next step should be turning this package into a component-only library.
Similar code reorganization can be applied to SCT_GeoModel too.

Subsystem readout geometry interfaces remain mostly unchanged, except for a minor
change in the InDet readout geometry interface (which was unavoidable).

Several non-GeoModel packages affected by this migration mostly required addition
of missing CLHEP includes.


Former-commit-id: 9cffebb1

Deprecating use of non-bound DataHandle, especially non-const.
Also adding const for materials.


Former-commit-id: 4c81c8a7

Former-commit-id: 2af8843e

Remove references to obsolete package CLIDSvc; header moved to AthenaKernel.
Also remove unneeded dependency on SGTools.



Former-commit-id: a8f03d1b

Headers from CLIDSvc have been moved to AthenaKernel.



Former-commit-id: 5d76402a

This reverts commit 6d8c8624 [formerly 8650a859].


Former-commit-id: bac9f4d6

Former-commit-id: 8650a859

Fix coverity warning: potential memory leak on error path.


Former-commit-id: d56d0904

Remove references to obsolete package CLIDSvc.


Former-commit-id: a08d7fe6

coverity: possible null pointer dereference.

Former-commit-id: e9ee2aef

Coverity warning: possible null pointer dereference.
Just remove the null test here; we're not anyway doing anything useful
if it fails.


Former-commit-id: 030f1040

dropped the obsolete reference to Database/AthenaPOOL/RDBAccessSvc
from Projects/AthSimulation/package_filters.txt


Former-commit-id: 7ce3b1e0

Private data members should start with m_.
Identifiers should not start with underscores.


Former-commit-id: 11f59468

This reverts commit cdf718bf [formerly 9d67f5b8].


Former-commit-id: 008b62d0

This reverts commit 9156e32ac88ebae9bf90ce0c19a40933a6fb097e [formerly f73d17d3f46239034e6ab596ed09b9aa922f4ad5], reversing
changes made to 27d106d1c78657e78ee8e51365e28572176cf185 [formerly 8f2f51887120a7b1cc2c7aff8c3101d0e69d4a2d].


Former-commit-id: 9d67f5b8a4078841f852715040f5a05bd7efc41b