From 5a7fdc02720672587ba716f30b992cd04be7f304 Mon Sep 17 00:00:00 2001 From: Dave Casper <dcasper@uci.edu> Date: Mon, 18 Nov 2019 09:25:05 -0800 Subject: [PATCH] Conform with ATLAS changes --- Control/CalypsoCommon/CMakeLists.txt | 1 - .../FaserGeoModel/python/SCTGMConfig.py | 3 +-- .../GeoModel/GeoModelSvc/CMakeLists.txt | 2 -- .../GeoModel/GeoModelSvc/src/GeoModelSvc.cxx | 3 +-- .../IdDictDetDescrCnv/CMakeLists.txt | 2 -- .../src/FaserIDDetDescrCnv.cxx | 3 +-- .../VetoG4_SD/src/VetoSensorSDTool.cxx | 2 +- .../ScintG4/VetoG4_SD/src/VetoSensorSDTool.h | 2 +- .../G4Faser/G4FaserInterfaces/CMakeLists.txt | 23 ------------------- Simulation/G4Utilities/Geo2G4/CMakeLists.txt | 2 +- .../src/FaserSctSensorSDTool.cxx | 2 +- .../FaserSCT_G4_SD/src/FaserSctSensorSDTool.h | 2 +- graphics/VTI12/VTI12Algs/CMakeLists.txt | 1 - .../VTI12GuideLineSystems/CMakeLists.txt | 2 +- .../VTI12Utils/src/SbPolyhedrizeAction.cxx | 2 +- 15 files changed, 10 insertions(+), 42 deletions(-) delete mode 100644 Simulation/G4Faser/G4FaserInterfaces/CMakeLists.txt diff --git a/Control/CalypsoCommon/CMakeLists.txt b/Control/CalypsoCommon/CMakeLists.txt index 3b99b49d..52a305d7 100644 --- a/Control/CalypsoCommon/CMakeLists.txt +++ b/Control/CalypsoCommon/CMakeLists.txt @@ -13,7 +13,6 @@ atlas_depends_on_subdirs( PRIVATE find_package( six ) # Install files from the package: -atlas_install_headers( CalypsoCommon ) atlas_install_python_modules( python/*.py python/Utils ) #atlas_install_joboptions( share/Preparation.py share/Execution.py share/Atlas.UnixStandardJob.py test/*.py # share/zeroJO.py share/Atlas_Gen.UnixStandardJob.py share/MemTraceInclude.py share/runbatch.py) diff --git a/DetectorDescription/GeoModel/FaserGeoModel/python/SCTGMConfig.py b/DetectorDescription/GeoModel/FaserGeoModel/python/SCTGMConfig.py index 81803ece..faae2c0d 100644 --- a/DetectorDescription/GeoModel/FaserGeoModel/python/SCTGMConfig.py +++ b/DetectorDescription/GeoModel/FaserGeoModel/python/SCTGMConfig.py @@ -3,16 +3,15 @@ # from AthenaConfiguration.ComponentAccumulator import ComponentAccumulator +from FaserSCT_GeoModel.FaserSCT_GeoModelConfig import FaserSCT_GeometryCfg def SctGeometryCfg (flags): acc = ComponentAccumulator() - from FaserSCT_GeoModel.FaserSCT_GeoModelConfig import FaserSCT_GeometryCfg acc.merge(FaserSCT_GeometryCfg( flags )) return acc if __name__ == "__main__": - import os from AthenaCommon.Logging import log from AthenaCommon.Constants import DEBUG from AthenaCommon.Configurable import Configurable diff --git a/DetectorDescription/GeoModel/GeoModelSvc/CMakeLists.txt b/DetectorDescription/GeoModel/GeoModelSvc/CMakeLists.txt index 48678911..d4ba92d1 100644 --- a/DetectorDescription/GeoModel/GeoModelSvc/CMakeLists.txt +++ b/DetectorDescription/GeoModel/GeoModelSvc/CMakeLists.txt @@ -7,7 +7,6 @@ atlas_subdir( GeoModelSvc ) # Declare the package's dependencies: atlas_depends_on_subdirs( PRIVATE - Control/CxxUtils Control/AthenaBaseComps Control/AthenaKernel Control/SGTools @@ -39,6 +38,5 @@ atlas_add_component( GeoModelSvc LINK_LIBRARIES ${Boost_LIBRARIES} ${CORAL_LIBRARIES} ${GEOMODEL_LIBRARIES} AthenaBaseComps CxxUtils AthenaKernel SGTools StoreGateLib SGtests GeoModelUtilities GeoModelFaserUtilities GeoModelDBManager GeoWrite EventInfo GaudiKernel ) # Install files from the package: -atlas_install_headers( GeoModelSvc ) #atlas_install_joboptions( share/*.py ) diff --git a/DetectorDescription/GeoModel/GeoModelSvc/src/GeoModelSvc.cxx b/DetectorDescription/GeoModel/GeoModelSvc/src/GeoModelSvc.cxx index 831d20e6..2278e5ab 100644 --- a/DetectorDescription/GeoModel/GeoModelSvc/src/GeoModelSvc.cxx +++ b/DetectorDescription/GeoModel/GeoModelSvc/src/GeoModelSvc.cxx @@ -18,7 +18,6 @@ #include "GaudiKernel/ISvcLocator.h" #include "GaudiKernel/IConversionSvc.h" #include "GaudiKernel/SystemOfUnits.h" -#include "CxxUtils/make_unique.h" #include "RDBAccessSvc/IRDBAccessSvc.h" #include "RDBAccessSvc/IRDBRecordset.h" @@ -337,7 +336,7 @@ StatusCode GeoModelSvc::geoInit() int mem,cpu; std::unique_ptr<std::ofstream> geoModelStats; if(m_statisticsToFile) { - geoModelStats = CxxUtils::make_unique<std::ofstream>("GeoModelStatistics"); + geoModelStats = std::make_unique<std::ofstream>("GeoModelStatistics"); *geoModelStats << "Detector Configuration flag = " << m_FaserVersion << std::endl; } diff --git a/DetectorDescription/IdDictDetDescrCnv/CMakeLists.txt b/DetectorDescription/IdDictDetDescrCnv/CMakeLists.txt index 460d4194..49e87e4e 100644 --- a/DetectorDescription/IdDictDetDescrCnv/CMakeLists.txt +++ b/DetectorDescription/IdDictDetDescrCnv/CMakeLists.txt @@ -8,7 +8,6 @@ atlas_subdir( IdDictDetDescrCnv ) # Declare the package's dependencies: atlas_depends_on_subdirs( PRIVATE Control/AthenaBaseComps - Control/CxxUtils Control/StoreGate Control/AthenaKernel Database/RDBAccessSvc @@ -33,5 +32,4 @@ atlas_add_component( IdDictDetDescrCnv LINK_LIBRARIES ${Boost_LIBRARIES} ${CORAL_LIBRARIES} AthenaBaseComps CxxUtils AthenaKernel StoreGateLib SGtests FaserDetDescr DetDescrCnvSvcLib GeoModelUtilities GeoModelFaserUtilities IdDictDetDescr IdDictParser Identifier GaudiKernel ) # Install files from the package: -atlas_install_headers( IdDictDetDescrCnv ) atlas_install_joboptions( share/*.py ) diff --git a/DetectorDescription/IdDictDetDescrCnv/src/FaserIDDetDescrCnv.cxx b/DetectorDescription/IdDictDetDescrCnv/src/FaserIDDetDescrCnv.cxx index dabb566b..c2468e6b 100644 --- a/DetectorDescription/IdDictDetDescrCnv/src/FaserIDDetDescrCnv.cxx +++ b/DetectorDescription/IdDictDetDescrCnv/src/FaserIDDetDescrCnv.cxx @@ -19,7 +19,6 @@ #include "IdDictDetDescr/IdDictManager.h" #include "FaserDetDescr/FaserDetectorID.h" -#include "CxxUtils/make_unique.h" //<<<<<< PRIVATE DEFINES >>>>>> //<<<<<< PRIVATE CONSTANTS >>>>>> @@ -132,7 +131,7 @@ FaserIDDetDescrCnv::createObj(IOpaqueAddress* pAddr, DataObject*& pObj) // create the helper - auto faser_id = CxxUtils::make_unique<FaserDetectorID>(); + auto faser_id = std::make_unique<FaserDetectorID>(); faser_id->setMessageSvc(msgSvc()); if (idDictMgr->initializeHelper(*faser_id)) { diff --git a/Scintillator/ScintG4/VetoG4_SD/src/VetoSensorSDTool.cxx b/Scintillator/ScintG4/VetoG4_SD/src/VetoSensorSDTool.cxx index bd19e951..c880dc21 100644 --- a/Scintillator/ScintG4/VetoG4_SD/src/VetoSensorSDTool.cxx +++ b/Scintillator/ScintG4/VetoG4_SD/src/VetoSensorSDTool.cxx @@ -24,7 +24,7 @@ VetoSensorSDTool::VetoSensorSDTool(const std::string& type, const std::string& n //....oooOO0OOooo........oooOO0OOooo........oooOO0OOooo........oooOO0OOooo...... -G4VSensitiveDetector* VetoSensorSDTool::makeSD() +G4VSensitiveDetector* VetoSensorSDTool::makeSD() const { ATH_MSG_DEBUG( "Creating Veto SD: " << name() ); return new VetoSensorSD(name(), m_outputCollectionNames[0]); diff --git a/Scintillator/ScintG4/VetoG4_SD/src/VetoSensorSDTool.h b/Scintillator/ScintG4/VetoG4_SD/src/VetoSensorSDTool.h index 33751c0b..8a139205 100644 --- a/Scintillator/ScintG4/VetoG4_SD/src/VetoSensorSDTool.h +++ b/Scintillator/ScintG4/VetoG4_SD/src/VetoSensorSDTool.h @@ -30,7 +30,7 @@ class VetoSensorSDTool : public SensitiveDetectorBase protected: // Make me an SD! - G4VSensitiveDetector* makeSD() override final; + G4VSensitiveDetector* makeSD() const override final; }; #endif //VETOG4_SD_VETOSENSORSDTOOL_H diff --git a/Simulation/G4Faser/G4FaserInterfaces/CMakeLists.txt b/Simulation/G4Faser/G4FaserInterfaces/CMakeLists.txt deleted file mode 100644 index b2c9cfb4..00000000 --- a/Simulation/G4Faser/G4FaserInterfaces/CMakeLists.txt +++ /dev/null @@ -1,23 +0,0 @@ -################################################################################ -# Package: G4FaserInterfaces -################################################################################ - -# Declare the package name: -atlas_subdir( G4FaserInterfaces ) - -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - GaudiKernel ) - -# External dependencies: -find_package( CLHEP ) -find_package( Geant4 ) -find_package( XercesC ) - -# Component(s) in the package: -atlas_add_library( G4FaserInterfaces - PUBLIC_HEADERS G4FaserInterfaces - INCLUDE_DIRS ${GEANT4_INCLUDE_DIRS} ${XERCESC_INCLUDE_DIRS} ${CLHEP_INCLUDE_DIRS} - DEFINITIONS ${CLHEP_DEFINITIONS} - LINK_LIBRARIES ${GEANT4_LIBRARIES} ${XERCESC_LIBRARIES} ${CLHEP_LIBRARIES} GaudiKernel ) - diff --git a/Simulation/G4Utilities/Geo2G4/CMakeLists.txt b/Simulation/G4Utilities/Geo2G4/CMakeLists.txt index f30e1839..cc72f51f 100644 --- a/Simulation/G4Utilities/Geo2G4/CMakeLists.txt +++ b/Simulation/G4Utilities/Geo2G4/CMakeLists.txt @@ -32,7 +32,7 @@ find_package( GeoModel ) # Component(s) in the package: atlas_add_library( Geo2G4Lib src/*.cxx - PUBLIC_HEADERS Geo2G4 + NO_PUBLIC_HEADERS INCLUDE_DIRS ${XERCESC_INCLUDE_DIRS} ${GEANT4_INCLUDE_DIRS} ${ROOT_INCLUDE_DIRS} PRIVATE_INCLUDE_DIRS ${Boost_INCLUDE_DIRS} ${CLHEP_INCLUDE_DIRS} ${GEOMODEL_INCLUDE_DIRS} PRIVATE_DEFINITIONS ${CLHEP_DEFINITIONS} diff --git a/Tracker/TrackerG4/FaserSCT_G4_SD/src/FaserSctSensorSDTool.cxx b/Tracker/TrackerG4/FaserSCT_G4_SD/src/FaserSctSensorSDTool.cxx index b1b3e9dd..322784b6 100644 --- a/Tracker/TrackerG4/FaserSCT_G4_SD/src/FaserSctSensorSDTool.cxx +++ b/Tracker/TrackerG4/FaserSCT_G4_SD/src/FaserSctSensorSDTool.cxx @@ -24,7 +24,7 @@ FaserSctSensorSDTool::FaserSctSensorSDTool(const std::string& type, const std::s //....oooOO0OOooo........oooOO0OOooo........oooOO0OOooo........oooOO0OOooo...... -G4VSensitiveDetector* FaserSctSensorSDTool::makeSD() +G4VSensitiveDetector* FaserSctSensorSDTool::makeSD() const { ATH_MSG_DEBUG( "Creating FaserSCT SD: " << name() ); return new FaserSctSensorSD(name(), m_outputCollectionNames[0]); diff --git a/Tracker/TrackerG4/FaserSCT_G4_SD/src/FaserSctSensorSDTool.h b/Tracker/TrackerG4/FaserSCT_G4_SD/src/FaserSctSensorSDTool.h index 2f53305d..d006d5ef 100644 --- a/Tracker/TrackerG4/FaserSCT_G4_SD/src/FaserSctSensorSDTool.h +++ b/Tracker/TrackerG4/FaserSCT_G4_SD/src/FaserSctSensorSDTool.h @@ -30,7 +30,7 @@ class FaserSctSensorSDTool : public SensitiveDetectorBase protected: // Make me an SD! - G4VSensitiveDetector* makeSD() override final; + G4VSensitiveDetector* makeSD() const override final; }; #endif //FASERSCT_G4_SD_FASERSCTSENSORSDTOOL_H diff --git a/graphics/VTI12/VTI12Algs/CMakeLists.txt b/graphics/VTI12/VTI12Algs/CMakeLists.txt index 6c233e64..dcfca1d4 100644 --- a/graphics/VTI12/VTI12Algs/CMakeLists.txt +++ b/graphics/VTI12/VTI12Algs/CMakeLists.txt @@ -28,7 +28,6 @@ atlas_add_component( VTI12Algs PRIVATE_LINK_LIBRARIES VTI12Gui VP1UtilsBase ) # Install files from the package: -atlas_install_headers( VTI12Algs ) atlas_install_python_modules( python/*.py ) atlas_install_joboptions( share/*.py ) atlas_install_scripts( share/vti12 ) diff --git a/graphics/VTI12/VTI12Systems/VTI12GuideLineSystems/CMakeLists.txt b/graphics/VTI12/VTI12Systems/VTI12GuideLineSystems/CMakeLists.txt index 09c90e47..de63ebf7 100644 --- a/graphics/VTI12/VTI12Systems/VTI12GuideLineSystems/CMakeLists.txt +++ b/graphics/VTI12/VTI12Systems/VTI12GuideLineSystems/CMakeLists.txt @@ -27,7 +27,7 @@ set( CMAKE_AUTOMOC TRUE ) # Component(s) in the package: atlas_add_library( VTI12GuideLineSystems VTI12GuideLineSystems/*.h src/*.cxx - PUBLIC_HEADERS VP1GuideLineSystems + NO_PUBLIC_HEADERS INCLUDE_DIRS ${COIN3D_INCLUDE_DIRS} LINK_LIBRARIES ${COIN3D_LIBRARIES} GeoPrimitives VP1Base VTI12Utils Qt5::Core Qt5::Gui diff --git a/graphics/VTI12/VTI12Utils/src/SbPolyhedrizeAction.cxx b/graphics/VTI12/VTI12Utils/src/SbPolyhedrizeAction.cxx index b09115f0..1f4a4f83 100644 --- a/graphics/VTI12/VTI12Utils/src/SbPolyhedrizeAction.cxx +++ b/graphics/VTI12/VTI12Utils/src/SbPolyhedrizeAction.cxx @@ -55,7 +55,7 @@ void SbPolyhedrizeAction::handleShift(const GeoShapeShift *shift) #include <VP1HEPVis/SbRotation.h> //using doubles instead of floats. HEPVis::SbRotation rot(rotation(0,0),rotation(1,0),rotation(2,0),0, rotation(0,1),rotation(1,1),rotation(2,1),0, - rotation(0,2),rotation(1,2),rotation(2.2),0, + rotation(0,2),rotation(1,2),rotation(2,2),0, 0,0,0,1); //--- -- GitLab