Remove reference to obsolete header StoreGate.h.

Remove reference to obsolete header StoreGate.h.

Frank Berghaus authored 4 years ago and Adam Edward Barton committed 4 years ago

This has just laid dormant ever since the CMT->CMake migration. :-(

At the same time introduced some tests for the implicit copy-constructor
and copy-operator of xAOD::EventFormat.

Prior to this change log file diffs in unit tests compared the reference file to the current
output, rather than the other way around, so that lines in the log removed due to local
changes are marked with a `+` in the diff output and lines added to the log are marked with
a `-`.

This commit switches the order of comparison with the aim of making the output more intuitive.
See discussion in ATEAM-644.

For TagInfo changes.

Fixed most of the ctest failures introduced by !35242

- remove atlas_depends_on_subdirs
- update link and external dependencies

- remove ROOT dependency
- remove atlas_depends_on_subdirs
- update link dependencies

With athenaMT it is best to rely on the default (ROOT) signal handler in
CoreDumpSvc, which uses gdb to produce the stacktrace for all threads.
Setting `FatalHandler = 438` makes use of the `SealSignal` handler,
which only produces one trace. For production environments where the gdb
traces are not needed, one should override the setting in the transform
or AMI tag as is already done today.

Relates to ATEAM-629.

Use explict cast for int->float conversion that loses precision.

Fix test failures in master.

SealSvc does not have any useful function any more

Frank Berghaus authored 4 years ago and Walter Lampl committed 4 years ago

The ByteStreamEventStorageInput service provided a legacy job option called FullFileName to specify bytestream input. With the move to EventSelector this option was all but removed from ByteStreamEventInputSvc. The EventSelectorByteStream provieded the input option to specify input. It captured the FullFileName option of the InputSvc. This commit removes the relevant parts of the ByteStreamEventStorageInputSvc and EventSelectorByteStream. It also attempts to move all references in other job options (unless they specify being for an older release).

Library dependency fixes.

Library dependency fixes.

Change use of tbb:task_scheduler_init to global_control.
The former is now deprecated.

StautsCodeSvc is deprecated. Ignore all differences due to it.

Align the definition with CxxUtils to avoid compiler warning.

Copying an AttributeList is potentially not thread-safe.
Fix an instance of that here (which was also a memory leak!).

Remove some files that were testing old thinning.
The tests are not running now, and things which they reference have been
deleted.

A quoting fix in the comparison script in order to handle properly
spaces in a selection pattern.  Update reference file.

Also fix sensitivity to paths used to find data files.