Move GeoExporter to use the standalone GeoModelIO from AthenaExternals.

The GeoExporter package is used by the DumpGeo Athena algorithm to dump the GeoModel tree to a local SQLite file. Until now, it still used the old version of theGeoWrite package, the one still hosted under Athena for back-compatibility.

The GeoWrite package has been replaced by the GeoModelWrite package hosted under the standalone GeoModelIO package.

This MR move the GeoExporter package to use the AthenaExternals/GeoModelIO package instead of the old GeoWrite. It also removes the old packages from Athena (they have been replaced by packages in the standalone GeoModel).

added 1 commit

• 0eff924e - Removing old Athena packages, now replaced and extended by standalone GeoModelIO

Compare with previous version

unmarked as a Work In Progress

changed the description

This merge request affects 6 packages:

• DetectorDescription/GeoModel/GeoModelStandalone/GeoExporter
• DetectorDescription/GeoModel/GeoModelStandalone/GeoModelDBManager
• DetectorDescription/GeoModel/GeoModelStandalone/GeoModelErrorHandler
• DetectorDescription/GeoModel/GeoModelStandalone/GeoRead
• DetectorDescription/GeoModel/GeoModelStandalone/GeoWrite
• DetectorDescription/GeoModel/GeoModelStandalone/TFPersistification

Adding @nkoehler ,@rbianchi as watchers

added Geometry master review-pending-level-1 labels

CI Result SUCCESS (hash 0eff924e)

	Athena	AthSimulation	AnalysisBase	AthGeneration
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
AnalysisBase: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 16561]

added review-approved label and removed review-pending-level-1 label

The CI is all green, no warnings.

Also, a note: the changes here only affect DumpGeo and GeoExporter; no other Athena packages should be affected at this point. (I'll push another MR later with the migration to the new GeoModel monorepo)

Best, -- Ric.

changed the description

mentioned in commit 78ed425a

merged

added sweep:ignore label