From 167f75d5599686ae9d8aab1b18ed4bdc4d54a0d7 Mon Sep 17 00:00:00 2001 From: Frank Winklmeier <fwinkl@cern> Date: Wed, 21 Oct 2020 16:00:20 +0200 Subject: [PATCH 1/8] InDetRegionSelector: cmake cleanup - remove `atlas_depends_on_subdirs` and fix link dependencies - remove unused CLHEP header and dependency - remove recursive include --- .../InDetRegionSelector/CMakeLists.txt | 29 ++----------------- .../src/InDetRegionSelectorLUT.cxx | 4 +-- .../src/TRT_RegSelCondAlg.cxx | 1 - .../src/TRT_RegSelCondAlg.h | 1 - 4 files changed, 3 insertions(+), 32 deletions(-) diff --git a/InnerDetector/InDetDetDescr/InDetRegionSelector/CMakeLists.txt b/InnerDetector/InDetDetDescr/InDetRegionSelector/CMakeLists.txt index fec15c73bdf6..be8a0efb9095 100644 --- a/InnerDetector/InDetDetDescr/InDetRegionSelector/CMakeLists.txt +++ b/InnerDetector/InDetDetDescr/InDetRegionSelector/CMakeLists.txt @@ -1,38 +1,13 @@ -################################################################################ -# Package: InDetRegionSelector -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( InDetRegionSelector ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaBaseComps - DetectorDescription/RegSelLUT - GaudiKernel - InnerDetector/InDetConditions/PixelConditionsData - InnerDetector/InDetDetDescr/SCT_Cabling - PRIVATE - Control/AthenaKernel - DetectorDescription/Identifier - DetectorDescription/IRegionSelector - InnerDetector/InDetDetDescr/InDetIdentifier - InnerDetector/InDetDetDescr/InDetReadoutGeometry - InnerDetector/InDetDetDescr/TRT_ReadoutGeometry - InnerDetector/InDetDetDescr/TRT_Cabling ) - -# External dependencies: -find_package( CLHEP ) -find_package( tdaq-common ) - # Component(s) in the package: atlas_add_component( InDetRegionSelector src/*.cxx src/components/*.cxx - INCLUDE_DIRS ${TDAQ-COMMON_INCLUDE_DIRS} ${CLHEP_INCLUDE_DIRS} - LINK_LIBRARIES ${TDAQ-COMMON_LIBRARIES} ${CLHEP_LIBRARIES} AthenaBaseComps RegSelLUT GaudiKernel PixelConditionsData SCT_CablingLib Identifier InDetIdentifier InDetReadoutGeometry TRT_ReadoutGeometry TRT_CablingLib AthenaKernel ) + LINK_LIBRARIES AthenaBaseComps RegSelLUT GaudiKernel PixelConditionsData SCT_CablingLib Identifier InDetIdentifier InDetReadoutGeometry IRegionSelector StoreGateLib TRT_ReadoutGeometry TRT_CablingLib ) # Install files from the package: -atlas_install_headers( InDetRegionSelector ) atlas_install_joboptions( share/*.py ) - diff --git a/InnerDetector/InDetDetDescr/InDetRegionSelector/src/InDetRegionSelectorLUT.cxx b/InnerDetector/InDetDetDescr/InDetRegionSelector/src/InDetRegionSelectorLUT.cxx index 194d06551246..02637cdc1f59 100755 --- a/InnerDetector/InDetDetDescr/InDetRegionSelector/src/InDetRegionSelectorLUT.cxx +++ b/InnerDetector/InDetDetDescr/InDetRegionSelector/src/InDetRegionSelectorLUT.cxx @@ -4,9 +4,7 @@ #include "InDetRegionSelector/InDetRegionSelectorLUT.h" -#include "CLHEP/Units/SystemOfUnits.h" - -#include "RegSelLUT/RegSelSiLUT.h" +#include "RegSelLUT/RegSelSiLUT.h" #include <vector> #include <string> diff --git a/InnerDetector/InDetDetDescr/InDetRegionSelector/src/TRT_RegSelCondAlg.cxx b/InnerDetector/InDetDetDescr/InDetRegionSelector/src/TRT_RegSelCondAlg.cxx index 9b85c6fa0826..60014c6353a6 100644 --- a/InnerDetector/InDetDetDescr/InDetRegionSelector/src/TRT_RegSelCondAlg.cxx +++ b/InnerDetector/InDetDetDescr/InDetRegionSelector/src/TRT_RegSelCondAlg.cxx @@ -22,7 +22,6 @@ #include "InDetReadoutGeometry/SiDetectorElement.h" #include "GaudiKernel/MsgStream.h" -#include "CLHEP/Units/SystemOfUnits.h" #include "TRT_ReadoutGeometry/TRT_BarrelElement.h" #include "TRT_ReadoutGeometry/TRT_EndcapElement.h" diff --git a/InnerDetector/InDetDetDescr/InDetRegionSelector/src/TRT_RegSelCondAlg.h b/InnerDetector/InDetDetDescr/InDetRegionSelector/src/TRT_RegSelCondAlg.h index c86a0f719621..b78127fc7e3f 100755 --- a/InnerDetector/InDetDetDescr/InDetRegionSelector/src/TRT_RegSelCondAlg.h +++ b/InnerDetector/InDetDetDescr/InDetRegionSelector/src/TRT_RegSelCondAlg.h @@ -29,7 +29,6 @@ #include <string> -#include "TRT_RegSelCondAlg.h" -- GitLab From 9420e4955725979e26f26674ee3b74d7b375d2c5 Mon Sep 17 00:00:00 2001 From: Frank Winklmeier <fwinkl@cern> Date: Wed, 21 Oct 2020 16:01:34 +0200 Subject: [PATCH 2/8] InDetDetDescr: cmake cleanup Remove `atlas_depends_on_subdirs` and update link dependencies. Enable flake8 where applicable. --- .../InDetDetDescr/BCM_GeoModel/CMakeLists.txt | 23 +--------- .../InDetDetDescr/BLM_GeoModel/CMakeLists.txt | 22 +-------- .../InDetGeoModelUtils/CMakeLists.txt | 26 ++--------- .../InDetIdDictFiles/CMakeLists.txt | 4 +- .../InDetIdentifier/CMakeLists.txt | 26 +++-------- .../InDetReadoutGeometry/CMakeLists.txt | 33 ++----------- .../InDetServMatGeoModel/CMakeLists.txt | 30 +----------- .../InDetTrackingGeometry/CMakeLists.txt | 36 ++------------- .../InDetDetDescr/PixelCabling/CMakeLists.txt | 34 ++++---------- .../PixelGeoModel/CMakeLists.txt | 46 ++++--------------- .../python/PixelGeoModelOldConfig.py | 1 - .../PixelReadoutGeometry/CMakeLists.txt | 37 ++------------- .../InDetDetDescr/SCT_Cabling/CMakeLists.txt | 38 ++++----------- .../InDetDetDescr/SCT_GeoModel/CMakeLists.txt | 40 ++++------------ .../SCT_ModuleDistortions/CMakeLists.txt | 37 ++++----------- .../SCT_ReadoutGeometry/CMakeLists.txt | 33 ++----------- .../StripGeoModelXml/CMakeLists.txt | 42 ++--------------- .../InDetDetDescr/TRT_Cabling/CMakeLists.txt | 18 +------- .../InDetDetDescr/TRT_GeoModel/CMakeLists.txt | 44 ++++-------------- .../TRT_ReadoutGeometry/CMakeLists.txt | 35 ++------------ 20 files changed, 95 insertions(+), 510 deletions(-) diff --git a/InnerDetector/InDetDetDescr/BCM_GeoModel/CMakeLists.txt b/InnerDetector/InDetDetDescr/BCM_GeoModel/CMakeLists.txt index e644836675f2..853f83c2901a 100644 --- a/InnerDetector/InDetDetDescr/BCM_GeoModel/CMakeLists.txt +++ b/InnerDetector/InDetDetDescr/BCM_GeoModel/CMakeLists.txt @@ -1,32 +1,13 @@ -################################################################################ -# Package: BCM_GeoModel -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( BCM_GeoModel ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaBaseComps - Control/AthenaKernel - DetectorDescription/GeoModel/GeoModelInterfaces - GaudiKernel - PRIVATE - Database/RDBAccessSvc - DetectorDescription/GeoModel/GeoModelUtilities ) - # External dependencies: -find_package( CORAL COMPONENTS CoralBase CoralKernel RelationalAccess ) -find_package( Eigen ) find_package( GeoModelCore ) # Component(s) in the package: atlas_add_component( BCM_GeoModel src/*.cxx src/components/*.cxx - INCLUDE_DIRS ${CORAL_INCLUDE_DIRS} - LINK_LIBRARIES ${CORAL_LIBRARIES} ${GEOMODELCORE_LIBRARIES} AthenaBaseComps AthenaKernel GaudiKernel GeoModelUtilities RDBAccessSvcLib ) - -# Install files from the package: -atlas_install_headers( BCM_GeoModel ) - + LINK_LIBRARIES ${GEOMODELCORE_LIBRARIES} AthenaBaseComps AthenaKernel GaudiKernel GeoModelInterfaces GeoModelUtilities RDBAccessSvcLib ) diff --git a/InnerDetector/InDetDetDescr/BLM_GeoModel/CMakeLists.txt b/InnerDetector/InDetDetDescr/BLM_GeoModel/CMakeLists.txt index d694cbdfdf2a..f3dbccbdb6f2 100644 --- a/InnerDetector/InDetDetDescr/BLM_GeoModel/CMakeLists.txt +++ b/InnerDetector/InDetDetDescr/BLM_GeoModel/CMakeLists.txt @@ -1,32 +1,14 @@ -################################################################################ -# Package: BLM_GeoModel -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( BLM_GeoModel ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaBaseComps - Control/AthenaKernel - DetectorDescription/GeoModel/GeoModelInterfaces - GaudiKernel - PRIVATE - Database/RDBAccessSvc - DetectorDescription/GeoModel/GeoModelUtilities ) - # External dependencies: -find_package( CORAL COMPONENTS CoralBase CoralKernel RelationalAccess ) find_package( GeoModelCore ) # Component(s) in the package: atlas_add_component( BLM_GeoModel src/*.cxx src/components/*.cxx - INCLUDE_DIRS ${CORAL_INCLUDE_DIRS} - LINK_LIBRARIES ${CORAL_LIBRARIES} ${GEOMODELCORE_LIBRARIES} AthenaBaseComps AthenaKernel GaudiKernel GeoModelUtilities + LINK_LIBRARIES ${GEOMODELCORE_LIBRARIES} AthenaBaseComps AthenaKernel GaudiKernel GeoModelInterfaces GeoModelUtilities PRIVATE_LINK_LIBRARIES RDBAccessSvcLib ) - -# Install files from the package: -atlas_install_headers( BLM_GeoModel ) - diff --git a/InnerDetector/InDetDetDescr/InDetGeoModelUtils/CMakeLists.txt b/InnerDetector/InDetDetDescr/InDetGeoModelUtils/CMakeLists.txt index e47c4885b376..a1f3f62abf14 100644 --- a/InnerDetector/InDetDetDescr/InDetGeoModelUtils/CMakeLists.txt +++ b/InnerDetector/InDetDetDescr/InDetGeoModelUtils/CMakeLists.txt @@ -1,34 +1,14 @@ -################################################################################ -# Package: InDetGeoModelUtils -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( InDetGeoModelUtils ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaKernel - Control/CxxUtils - Database/RDBAccessSvc - GaudiKernel - PRIVATE - Control/StoreGate - DetectorDescription/GeoModel/GeoModelInterfaces - DetectorDescription/GeoModel/GeoModelUtilities - DetectorDescription/GeoPrimitives - DetectorDescription/GeometryDBSvc ) - # External dependencies: -find_package( CLHEP ) -find_package( CORAL COMPONENTS CoralBase CoralKernel RelationalAccess ) find_package( GeoModelCore ) # Component(s) in the package: atlas_add_library( InDetGeoModelUtils src/*.cxx PUBLIC_HEADERS InDetGeoModelUtils - INCLUDE_DIRS ${CORAL_INCLUDE_DIRS} ${CLHEP_INCLUDE_DIRS} - DEFINITIONS ${CLHEP_DEFINITIONS} - LINK_LIBRARIES ${CORAL_LIBRARIES} ${CLHEP_LIBRARIES} ${GEOMODELCORE_LIBRARIES} AthenaKernel CxxUtils GaudiKernel StoreGateLib SGtests RDBAccessSvcLib GeoPrimitives - PRIVATE_LINK_LIBRARIES GeoModelUtilities GeometryDBSvcLib ) - + LINK_LIBRARIES ${GEOMODELCORE_LIBRARIES} AthenaKernel CxxUtils GaudiKernel RDBAccessSvcLib GeoPrimitives + PRIVATE_LINK_LIBRARIES GeoModelInterfaces GeoModelUtilities GeometryDBSvcLib StoreGateLib ) diff --git a/InnerDetector/InDetDetDescr/InDetIdDictFiles/CMakeLists.txt b/InnerDetector/InDetDetDescr/InDetIdDictFiles/CMakeLists.txt index 09623edec72f..bf31688247d6 100644 --- a/InnerDetector/InDetDetDescr/InDetIdDictFiles/CMakeLists.txt +++ b/InnerDetector/InDetDetDescr/InDetIdDictFiles/CMakeLists.txt @@ -1,6 +1,4 @@ -################################################################################ -# Package: InDetIdDictFiles -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( InDetIdDictFiles ) diff --git a/InnerDetector/InDetDetDescr/InDetIdentifier/CMakeLists.txt b/InnerDetector/InDetDetDescr/InDetIdentifier/CMakeLists.txt index 92a7be8d07ee..3ac014a971a8 100644 --- a/InnerDetector/InDetDetDescr/InDetIdentifier/CMakeLists.txt +++ b/InnerDetector/InDetDetDescr/InDetIdentifier/CMakeLists.txt @@ -1,22 +1,9 @@ -################################################################################ -# Package: InDetIdentifier -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( InDetIdentifier ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaKernel - DetectorDescription/AtlasDetDescr - DetectorDescription/IdDict - DetectorDescription/Identifier - PRIVATE - DetectorDescription/IdDictParser - GaudiKernel ) - # External dependencies: -find_package( ROOT COMPONENTS Core Tree MathCore Hist RIO pthread ) find_package( Boost COMPONENTS unit_test_framework ) # Component(s) in the package: @@ -26,21 +13,20 @@ atlas_add_library( InDetIdentifier src/TRT_ID.cxx src/SiliconID.cxx PUBLIC_HEADERS InDetIdentifier - PRIVATE_INCLUDE_DIRS ${ROOT_INCLUDE_DIRS} LINK_LIBRARIES AthenaKernel AtlasDetDescr IdDict Identifier - PRIVATE_LINK_LIBRARIES ${ROOT_LIBRARIES} IdDictParser GaudiKernel ) + PRIVATE_LINK_LIBRARIES GaudiKernel ) atlas_add_dictionary( InDetIdentifierDict InDetIdentifier/InDetIdentifierDict.h InDetIdentifier/selection.xml - INCLUDE_DIRS ${ROOT_INCLUDE_DIRS} - LINK_LIBRARIES ${ROOT_LIBRARIES} AtlasDetDescr IdDict Identifier IdDictParser GaudiKernel InDetIdentifier ) - + LINK_LIBRARIES InDetIdentifier ) + +# Tests in the package: atlas_add_test( SCT_ID_test SOURCES test/SCT_ID_test.cxx INCLUDE_DIRS ${Boost_INCLUDE_DIRS} - LINK_LIBRARIES ${Boost_LIBRARIES} AtlasDetDescr IdDict Identifier IdDictParser GaudiKernel InDetIdentifier + LINK_LIBRARIES ${Boost_LIBRARIES} IdDictParser InDetIdentifier POST_EXEC_SCRIPT "nopost.sh" ) set_target_properties( InDetIdentifier_SCT_ID_test PROPERTIES ENABLE_EXPORTS True ) diff --git a/InnerDetector/InDetDetDescr/InDetReadoutGeometry/CMakeLists.txt b/InnerDetector/InDetDetDescr/InDetReadoutGeometry/CMakeLists.txt index 49d45b2dd17f..a3abb437c413 100644 --- a/InnerDetector/InDetDetDescr/InDetReadoutGeometry/CMakeLists.txt +++ b/InnerDetector/InDetDetDescr/InDetReadoutGeometry/CMakeLists.txt @@ -1,42 +1,17 @@ -################################################################################ -# Package: InDetReadoutGeometry -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( InDetReadoutGeometry ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaBaseComps - Control/AthenaKernel - Control/StoreGate - DetectorDescription/AtlasDetDescr - DetectorDescription/GeoModel/GeoModelUtilities - DetectorDescription/GeoPrimitives - DetectorDescription/Identifier - GaudiKernel - InnerDetector/InDetConditions/InDetCondTools - InnerDetector/InDetDetDescr/InDetIdentifier - Tracking/TrkDetDescr/TrkDetElementBase - Tracking/TrkDetDescr/TrkSurfaces - Tracking/TrkEvent/TrkEventPrimitives - DetectorDescription/DetDescrCond/DetDescrConditions - PRIVATE - Database/AthenaPOOL/AthenaPoolUtilities - DetectorDescription/IdDictDetDescr - InnerDetector/InDetConditions/TRT_ConditionsData ) - # External dependencies: find_package( CLHEP ) -find_package( Eigen ) find_package( GeoModelCore ) # Component(s) in the package: atlas_add_library( InDetReadoutGeometry src/*.cxx PUBLIC_HEADERS InDetReadoutGeometry - INCLUDE_DIRS ${CLHEP_INCLUDE_DIRS} ${EIGEN_INCLUDE_DIRS} + INCLUDE_DIRS ${CLHEP_INCLUDE_DIRS} DEFINITIONS ${CLHEP_DEFINITIONS} - LINK_LIBRARIES ${CLHEP_LIBRARIES} ${EIGEN_LIBRARIES} ${GEOMODELCORE_LIBRARIES} AthenaKernel AtlasDetDescr GeoModelUtilities GeoPrimitives Identifier GaudiKernel InDetIdentifier TrkDetElementBase TrkSurfaces TrkEventPrimitives StoreGateLib SGtests AthenaBaseComps DetDescrConditions InDetCondTools - PRIVATE_LINK_LIBRARIES AthenaPoolUtilities IdDictDetDescr TRT_ConditionsData ) - + LINK_LIBRARIES ${CLHEP_LIBRARIES} ${GEOMODELCORE_LIBRARIES} AthenaKernel AtlasDetDescr CxxUtils GeoPrimitives Identifier GaudiKernel InDetIdentifier TrkDetElementBase TrkSurfaces TrkEventPrimitives StoreGateLib DetDescrConditions InDetCondTools + PRIVATE_LINK_LIBRARIES AthenaBaseComps AthenaPoolUtilities IdDictDetDescr GeoModelUtilities ) diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/CMakeLists.txt b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/CMakeLists.txt index f8de9f5d4021..ad08d99dcb43 100644 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/CMakeLists.txt +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/CMakeLists.txt @@ -1,39 +1,13 @@ -################################################################################ -# Package: InDetServMatGeoModel -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( InDetServMatGeoModel ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaBaseComps - Control/AthenaKernel - Control/CxxUtils - Database/RDBAccessSvc - DetectorDescription/GeoPrimitives - DetectorDescription/GeoModel/GeoModelUtilities - GaudiKernel - InnerDetector/InDetDetDescr/InDetGeoModelUtils - PRIVATE - Control/AthenaKernel - Control/SGTools - Control/StoreGate - DetectorDescription/GeoModel/GeoModelInterfaces - DetectorDescription/GeometryDBSvc ) - # External dependencies: -find_package( CORAL COMPONENTS CoralBase CoralKernel RelationalAccess ) -find_package( Eigen ) find_package( GeoModelCore ) # Component(s) in the package: atlas_add_component( InDetServMatGeoModel src/*.cxx src/components/*.cxx - INCLUDE_DIRS ${CORAL_INCLUDE_DIRS} - LINK_LIBRARIES ${CORAL_LIBRARIES} ${GEOMODELCORE_LIBRARIES} AthenaBaseComps AthenaKernel CxxUtils GeoModelUtilities GaudiKernel InDetGeoModelUtils SGTools StoreGateLib SGtests GeometryDBSvcLib ) - -# Install files from the package: -atlas_install_headers( InDetServMatGeoModel ) - + LINK_LIBRARIES ${GEOMODELCORE_LIBRARIES} AthenaBaseComps AthenaKernel CxxUtils GaudiKernel GeoModelInterfaces GeoModelUtilities GeoPrimitives GeometryDBSvcLib InDetGeoModelUtils RDBAccessSvcLib SGTools StoreGateLib ) diff --git a/InnerDetector/InDetDetDescr/InDetTrackingGeometry/CMakeLists.txt b/InnerDetector/InDetDetDescr/InDetTrackingGeometry/CMakeLists.txt index c64e06224ffd..70612b1af663 100644 --- a/InnerDetector/InDetDetDescr/InDetTrackingGeometry/CMakeLists.txt +++ b/InnerDetector/InDetDetDescr/InDetTrackingGeometry/CMakeLists.txt @@ -1,46 +1,18 @@ -################################################################################ -# Package: InDetTrackingGeometry -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( InDetTrackingGeometry ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaBaseComps - Control/CxxUtils - DetectorDescription/GeoPrimitives - GaudiKernel - InnerDetector/InDetDetDescr/InDetIdentifier - Tracking/TrkDetDescr/TrkDetDescrInterfaces - Tracking/TrkDetDescr/TrkDetDescrUtils - Tracking/TrkDetDescr/TrkGeometry - PRIVATE - AtlasGeometryCommon/BeamPipeGeoModel - AtlasGeometryCommon/SubDetectorEnvelopes - Control/StoreGate - DetectorDescription/Identifier - InnerDetector/InDetDetDescr/InDetReadoutGeometry - InnerDetector/InDetDetDescr/PixelReadoutGeometry - InnerDetector/InDetDetDescr/SCT_ReadoutGeometry - InnerDetector/InDetDetDescr/TRT_ReadoutGeometry - Tracking/TrkDetDescr/TrkDetDescrGeoModelCnv - Tracking/TrkDetDescr/TrkSurfaces - Tracking/TrkDetDescr/TrkVolumes ) - # External dependencies: -find_package( Eigen ) -find_package( ROOT COMPONENTS Core Tree MathCore Hist RIO pthread ) +find_package( Boost ) find_package( GeoModelCore ) # Component(s) in the package: atlas_add_component( InDetTrackingGeometry src/*.cxx src/components/*.cxx - INCLUDE_DIRS ${ROOT_INCLUDE_DIRS} ${EIGEN_INCLUDE_DIRS} - LINK_LIBRARIES ${ROOT_LIBRARIES} ${EIGEN_LIBRARIES} ${GEOMODELCORE_LIBRARIES} AthenaBaseComps CxxUtils GeoPrimitives GaudiKernel InDetIdentifier TrkDetDescrInterfaces TrkDetDescrUtils TrkGeometry BeamPipeGeoModelLib StoreGateLib SGtests Identifier InDetReadoutGeometry PixelReadoutGeometry SCT_ReadoutGeometry TRT_ReadoutGeometry TrkDetDescrGeoModelCnv TrkSurfaces TrkVolumes SubDetectorEnvelopesLib ) + INCLUDE_DIRS ${Boost_INCLUDE_DIRS} + LINK_LIBRARIES ${Boost_LIBRARIES} ${GEOMODELCORE_LIBRARIES} AthenaBaseComps CxxUtils GeoPrimitives GaudiKernel InDetIdentifier TrkDetDescrInterfaces TrkDetDescrUtils TrkGeometry BeamPipeGeoModelLib StoreGateLib Identifier InDetReadoutGeometry PixelReadoutGeometry SCT_ReadoutGeometry TRT_ReadoutGeometry TrkDetDescrGeoModelCnv TrkSurfaces TrkVolumes SubDetectorEnvelopesLib ) # Install files from the package: -atlas_install_headers( InDetTrackingGeometry ) atlas_install_python_modules( python/*.py POST_BUILD_CMD ${ATLAS_FLAKE8} ) - diff --git a/InnerDetector/InDetDetDescr/PixelCabling/CMakeLists.txt b/InnerDetector/InDetDetDescr/PixelCabling/CMakeLists.txt index 25c15aefc24e..bad773367aed 100644 --- a/InnerDetector/InDetDetDescr/PixelCabling/CMakeLists.txt +++ b/InnerDetector/InDetDetDescr/PixelCabling/CMakeLists.txt @@ -1,42 +1,24 @@ -################################################################################ -# Package: PixelCabling -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( PixelCabling ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( - PUBLIC - Control/AthenaKernel - GaudiKernel - PRIVATE - Control/AthenaBaseComps - Control/StoreGate - Database/AthenaPOOL/AthenaPoolUtilities - DetectorDescription/Identifier - InnerDetector/InDetDetDescr/InDetReadoutGeometry - InnerDetector/InDetDetDescr/PixelReadoutGeometry - InnerDetector/InDetDetDescr/InDetIdentifier - InnerDetector/InDetConditions/PixelConditionsData - Tools/PathResolver ) - # Component(s) in the package: atlas_add_library( PixelCablingLib PixelCabling/*.h INTERFACE PUBLIC_HEADERS PixelCabling - LINK_LIBRARIES GaudiKernel AthenaKernel ) + LINK_LIBRARIES GaudiKernel Identifier InDetIdentifier ) atlas_add_component( PixelCabling src/*.h src/*.cxx src/components/*.cxx - LINK_LIBRARIES GaudiKernel AthenaBaseComps SGTools StoreGateLib - AthenaPoolUtilities Identifier InDetIdentifier InDetReadoutGeometry PixelReadoutGeometry PixelConditionsData PathResolver PixelCablingLib ) + LINK_LIBRARIES AthenaBaseComps GaudiKernel InDetReadoutGeometry PixelCablingLib PixelReadoutGeometry StoreGateLib ) + +# Install files from the package: +atlas_install_python_modules( python/*.py POST_BUILD_CMD ${ATLAS_FLAKE8} ) +atlas_install_joboptions( share/*.py ) +# Test(s) in the package: atlas_add_test( PixelCablingConfigNew_test SCRIPT test/PixelCablingConfigNew_test.py PROPERTIES TIMEOUT 300 ) - -# Install files from the package: -atlas_install_python_modules( python/*.py ) -atlas_install_joboptions( share/*.py ) diff --git a/InnerDetector/InDetDetDescr/PixelGeoModel/CMakeLists.txt b/InnerDetector/InDetDetDescr/PixelGeoModel/CMakeLists.txt index a502cacda395..21ab313db3c7 100644 --- a/InnerDetector/InDetDetDescr/PixelGeoModel/CMakeLists.txt +++ b/InnerDetector/InDetDetDescr/PixelGeoModel/CMakeLists.txt @@ -1,56 +1,28 @@ -################################################################################ -# Package: PixelGeoModel -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( PixelGeoModel ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( - PUBLIC - Control/AthenaBaseComps - Control/AthenaKernel - Control/CxxUtils - Database/RDBAccessSvc - DetectorDescription/GeoModel/GeoModelInterfaces - DetectorDescription/GeoModel/GeoModelUtilities - DetectorDescription/Identifier - DetectorDescription/GeoPrimitives - GaudiKernel - InnerDetector/InDetDetDescr/InDetGeoModelUtils - InnerDetector/InDetDetDescr/InDetReadoutGeometry - InnerDetector/InDetDetDescr/PixelReadoutGeometry - PRIVATE - Control/SGTools - Control/StoreGate - DetectorDescription/DetDescrCond/DetDescrConditions - DetectorDescription/GeometryDBSvc - InnerDetector/InDetDetDescr/InDetIdentifier ) - # External dependencies: -find_package( Eigen ) find_package( GeoModelCore ) # Component(s) in the package: atlas_add_library( PixelGeoModelLib PixelGeoModel/*.h src/*.cxx PUBLIC_HEADERS PixelGeoModel - INCLUDE_DIRS ${EIGEN_INCLUDE_DIRS} - LINK_LIBRARIES ${EIGEN_LIBRARIES} ${GEOMODELCORE_LIBRARIES} AthenaBaseComps AthenaKernel CxxUtils - GeoModelUtilities Identifier GaudiKernel InDetGeoModelUtils - InDetReadoutGeometry PixelReadoutGeometry - PRIVATE_LINK_LIBRARIES SGTools StoreGateLib DetDescrConditions - InDetIdentifier GeometryDBSvcLib + LINK_LIBRARIES AthenaBaseComps GaudiKernel GeoModelUtilities GeometryDBSvcLib InDetGeoModelUtils + PRIVATE_LINK_LIBRARIES ${GEOMODELCORE_LIBRARIES} AthenaKernel CxxUtils DetDescrConditions GeoModelInterfaces GeoPrimitives Identifier InDetIdentifier InDetReadoutGeometry PixelReadoutGeometry RDBAccessSvcLib SGTools StoreGateLib PRIVATE_DEFINITIONS "-DGEOTORUS=1" ) atlas_add_component( PixelGeoModel src/components/*.cxx - LINK_LIBRARIES GaudiKernel PixelGeoModelLib GeometryDBSvcLib ) + LINK_LIBRARIES PixelGeoModelLib ) + +# Install files from the package: +atlas_install_python_modules( python/*.py POST_BUILD_CMD ${ATLAS_FLAKE8} ) +atlas_install_scripts( test/*.py ) +# Test(s) in the package: atlas_add_test( PixelGMConfig_test SCRIPT test/PixelGMConfig_test.py PROPERTIES TIMEOUT 300 ) - -# Install files from the package: -atlas_install_python_modules( python/*.py ) -atlas_install_scripts( test/*.py ) diff --git a/InnerDetector/InDetDetDescr/PixelGeoModel/python/PixelGeoModelOldConfig.py b/InnerDetector/InDetDetDescr/PixelGeoModel/python/PixelGeoModelOldConfig.py index b1bd5da0ab1e..be9854178a83 100644 --- a/InnerDetector/InDetDetDescr/PixelGeoModel/python/PixelGeoModelOldConfig.py +++ b/InnerDetector/InDetDetDescr/PixelGeoModel/python/PixelGeoModelOldConfig.py @@ -3,7 +3,6 @@ from AthenaCommon import CfgMgr def getPixelDetectorTool(name="PixelDetectorTool", **kwargs): - from AtlasGeoModel.InDetGMJobProperties import InDetGeometryFlags kwargs.setdefault("ServiceBuilderTool", "") return CfgMgr.PixelDetectorTool(name, **kwargs) diff --git a/InnerDetector/InDetDetDescr/PixelReadoutGeometry/CMakeLists.txt b/InnerDetector/InDetDetDescr/PixelReadoutGeometry/CMakeLists.txt index 88c461cdd223..d592ead53dce 100644 --- a/InnerDetector/InDetDetDescr/PixelReadoutGeometry/CMakeLists.txt +++ b/InnerDetector/InDetDetDescr/PixelReadoutGeometry/CMakeLists.txt @@ -1,43 +1,14 @@ -################################################################################ -# Package: InDetReadoutGeometry -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( PixelReadoutGeometry ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaBaseComps - Control/AthenaKernel - Control/StoreGate - DetectorDescription/AtlasDetDescr - DetectorDescription/GeoModel/GeoModelUtilities - DetectorDescription/GeoPrimitives - DetectorDescription/Identifier - GaudiKernel - InnerDetector/InDetConditions/InDetCondTools - InnerDetector/InDetDetDescr/InDetIdentifier - InnerDetector/InDetDetDescr/InDetReadoutGeometry - Tracking/TrkDetDescr/TrkDetElementBase - Tracking/TrkDetDescr/TrkSurfaces - Tracking/TrkEvent/TrkEventPrimitives - DetectorDescription/DetDescrCond/DetDescrConditions - PRIVATE - Database/AthenaPOOL/AthenaPoolUtilities - DetectorDescription/IdDictDetDescr) - # External dependencies: -find_package( CLHEP ) -find_package( Eigen ) find_package( GeoModelCore ) # Component(s) in the package: - atlas_add_library( PixelReadoutGeometry - src/*.c* + src/*.cxx PUBLIC_HEADERS PixelReadoutGeometry - INCLUDE_DIRS ${CLHEP_INCLUDE_DIRS} ${EIGEN_INCLUDE_DIRS} - DEFINITIONS ${CLHEP_DEFINITIONS} - LINK_LIBRARIES ${CLHEP_LIBRARIES} ${EIGEN_LIBRARIES} InDetReadoutGeometry AthenaKernel SGTools AtlasDetDescr GeoPrimitives Identifier GaudiKernel InDetIdentifier TrkDetElementBase TrkSurfaces TrkEventPrimitives StoreGateLib SGtests - PRIVATE_LINK_LIBRARIES AthenaPoolUtilities DetDescrConditions IdDictDetDescr) - + LINK_LIBRARIES ${GEOMODELCORE_LIBRARIES} AthenaKernel CxxUtils GeoPrimitives InDetIdentifier InDetReadoutGeometry TrkDetElementBase TrkSurfaces + PRIVATE_LINK_LIBRARIES AthenaBaseComps AthenaPoolUtilities DetDescrConditions Identifier StoreGateLib TrkEventPrimitives ) diff --git a/InnerDetector/InDetDetDescr/SCT_Cabling/CMakeLists.txt b/InnerDetector/InDetDetDescr/SCT_Cabling/CMakeLists.txt index 4b6efce09c20..7aa1cf2cde2b 100644 --- a/InnerDetector/InDetDetDescr/SCT_Cabling/CMakeLists.txt +++ b/InnerDetector/InDetDetDescr/SCT_Cabling/CMakeLists.txt @@ -1,45 +1,25 @@ -################################################################################ -# Package: SCT_Cabling -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( SCT_Cabling ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaBaseComps - DetectorDescription/Identifier - GaudiKernel - PRIVATE - Control/StoreGate - Database/AthenaPOOL/AthenaPoolUtilities - Database/CoraCool - InnerDetector/InDetDetDescr/InDetIdentifier - Tools/PathResolver ) - -# External dependencies: -find_package( COOL COMPONENTS CoolKernel CoolApplication ) -find_package( CORAL COMPONENTS CoralBase CoralKernel RelationalAccess ) -find_package( ROOT COMPONENTS Core Tree MathCore Hist RIO pthread ) - # Component(s) in the package: atlas_add_library( SCT_CablingLib src/*.cxx PUBLIC_HEADERS SCT_Cabling - PRIVATE_INCLUDE_DIRS ${COOL_INCLUDE_DIRS} ${ROOT_INCLUDE_DIRS} ${CORAL_INCLUDE_DIRS} - LINK_LIBRARIES AthenaBaseComps Identifier GaudiKernel StoreGateLib SGtests - PRIVATE_LINK_LIBRARIES ${COOL_LIBRARIES} ${ROOT_LIBRARIES} ${CORAL_LIBRARIES} AthenaPoolUtilities CoraCool InDetIdentifier PathResolver ) + LINK_LIBRARIES AthenaBaseComps AthenaKernel AthenaPoolUtilities GaudiKernel Identifier + PRIVATE_LINK_LIBRARIES InDetIdentifier PathResolver StoreGateLib ) atlas_add_component( SCT_Cabling src/components/*.cxx - INCLUDE_DIRS ${COOL_INCLUDE_DIRS} ${ROOT_INCLUDE_DIRS} ${CORAL_INCLUDE_DIRS} - LINK_LIBRARIES ${COOL_LIBRARIES} ${ROOT_LIBRARIES} ${CORAL_LIBRARIES} AthenaBaseComps Identifier GaudiKernel StoreGateLib SGtests AthenaPoolUtilities CoraCool InDetIdentifier PathResolver SCT_CablingLib ) + LINK_LIBRARIES SCT_CablingLib ) + +# Install files from the package: +atlas_install_joboptions( share/*.py ) +atlas_install_python_modules( python/*.py POST_BUILD_CMD ${ATLAS_FLAKE8} ) +# Test(s) in the package: atlas_add_test( TestSCT_CablingCfg SCRIPT python -m SCT_Cabling.TestSCT_CablingCfg PROPERTIES TIMEOUT 600 ENVIRONMENT THREADS=1 ) - -# Install files from the package: -atlas_install_joboptions( share/*.py ) -atlas_install_python_modules( python/*.py ) diff --git a/InnerDetector/InDetDetDescr/SCT_GeoModel/CMakeLists.txt b/InnerDetector/InDetDetDescr/SCT_GeoModel/CMakeLists.txt index b52eaaf9cfc9..86efac1d352d 100644 --- a/InnerDetector/InDetDetDescr/SCT_GeoModel/CMakeLists.txt +++ b/InnerDetector/InDetDetDescr/SCT_GeoModel/CMakeLists.txt @@ -1,46 +1,22 @@ -################################################################################ -# Package: SCT_GeoModel -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( SCT_GeoModel ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaKernel - Database/RDBAccessSvc - DetectorDescription/GeoModel/GeoModelUtilities - DetectorDescription/GeoPrimitives - GaudiKernel - InnerDetector/InDetDetDescr/InDetGeoModelUtils - InnerDetector/InDetDetDescr/InDetReadoutGeometry - InnerDetector/InDetDetDescr/SCT_ReadoutGeometry - PRIVATE - Control/SGTools - Control/StoreGate - Database/AthenaPOOL/AthenaPoolUtilities - DetectorDescription/DetDescrCond/DetDescrConditions - DetectorDescription/GeoModel/GeoModelInterfaces - DetectorDescription/GeometryDBSvc - DetectorDescription/Identifier - InnerDetector/InDetDetDescr/InDetIdentifier ) - # External dependencies: -find_package( CORAL COMPONENTS CoralBase CoralKernel RelationalAccess ) -find_package( Eigen ) find_package( GeoModelCore ) # Component(s) in the package: atlas_add_component( SCT_GeoModel - src/*.cxx + src/*.cxx src/components/*.cxx - INCLUDE_DIRS ${CORAL_INCLUDE_DIRS} - LINK_LIBRARIES ${CORAL_LIBRARIES} ${GEOMODELCORE_LIBRARIES} AthenaKernel GeoModelUtilities GaudiKernel InDetGeoModelUtils InDetReadoutGeometry SCT_ReadoutGeometry SGTools StoreGateLib SGtests AthenaPoolUtilities DetDescrConditions Identifier InDetIdentifier GeometryDBSvcLib ) + LINK_LIBRARIES ${GEOMODELCORE_LIBRARIES} AthenaKernel CxxUtils GeoModelInterfaces GeoModelUtilities GeoPrimitives GaudiKernel InDetGeoModelUtils InDetReadoutGeometry SCT_ReadoutGeometry SGTools StoreGateLib AthenaPoolUtilities DetDescrConditions Identifier InDetIdentifier GeometryDBSvcLib RDBAccessSvcLib ) + +# Install files from the package: +atlas_install_python_modules( python/*.py POST_BUILD_CMD ${ATLAS_FLAKE8} ) +atlas_install_scripts( test/*.py ) +# Test(s) in the package: atlas_add_test( SCT_GMConfig_test SCRIPT test/SCT_GMConfig_test.py PROPERTIES TIMEOUT 300 ) - -# Install files from the package: -atlas_install_python_modules( python/*.py ) -atlas_install_scripts( test/*.py ) diff --git a/InnerDetector/InDetDetDescr/SCT_ModuleDistortions/CMakeLists.txt b/InnerDetector/InDetDetDescr/SCT_ModuleDistortions/CMakeLists.txt index 017a2b1f54ab..8b00587effa4 100644 --- a/InnerDetector/InDetDetDescr/SCT_ModuleDistortions/CMakeLists.txt +++ b/InnerDetector/InDetDetDescr/SCT_ModuleDistortions/CMakeLists.txt @@ -1,29 +1,10 @@ -################################################################################ -# Package: SCT_ModuleDistortions -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( SCT_ModuleDistortions ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaKernel - DetectorDescription/GeoPrimitives - DetectorDescription/Identifier - GaudiKernel - InnerDetector/InDetDetDescr/InDetReadoutGeometry - Tracking/TrkEvent/TrkEventPrimitives - Tracking/TrkEvent/TrkParameters - PRIVATE - Control/AthenaBaseComps - Control/StoreGate - InnerDetector/InDetDetDescr/InDetIdentifier - Tools/PathResolver ) - # External dependencies: -find_package( CLHEP ) -find_package( Eigen ) -find_package( ROOT COMPONENTS Core Tree MathCore Hist RIO pthread ) +find_package( ROOT COMPONENTS Core Hist ) atlas_add_library( SCT_ModuleDistortionsLib SCT_ModuleDistortions/*.h @@ -31,19 +12,19 @@ atlas_add_library( SCT_ModuleDistortionsLib PUBLIC_HEADERS SCT_ModuleDistortions LINK_LIBRARIES GaudiKernel GeoPrimitives TrkParameters Identifier AthenaKernel ) - # Component(s) in the package: atlas_add_component( SCT_ModuleDistortions src/*.cxx src/components/*.cxx - INCLUDE_DIRS ${ROOT_INCLUDE_DIRS} ${CLHEP_INCLUDE_DIRS} ${EIGEN_INCLUDE_DIRS} - LINK_LIBRARIES ${ROOT_LIBRARIES} ${CLHEP_LIBRARIES} ${EIGEN_LIBRARIES} SCT_ModuleDistortionsLib InDetReadoutGeometry TrkEventPrimitives AthenaBaseComps StoreGateLib SGtests InDetIdentifier PathResolver ) + INCLUDE_DIRS ${ROOT_INCLUDE_DIRS} + LINK_LIBRARIES ${ROOT_LIBRARIES} AthenaBaseComps AthenaKernel GaudiKernel GeoPrimitives Identifier InDetIdentifier InDetReadoutGeometry PathResolver SCT_ModuleDistortionsLib TrkEventPrimitives TrkParameters ) + +# Install files from the package: +atlas_install_joboptions( share/*.py ) +atlas_install_runtime( share/*.txt ) +# Test(s) in the package: atlas_add_test( TestSCT_DistortionsTool SCRIPT athena.py --threads=5 SCT_ModuleDistortions/TestSCT_DistortionsTool.py PROPERTIES TIMEOUT 600 ENVIRONMENT THREADS=5 ) - -# Install files from the package: -atlas_install_joboptions( share/*.py ) -atlas_install_runtime( share/*.txt ) diff --git a/InnerDetector/InDetDetDescr/SCT_ReadoutGeometry/CMakeLists.txt b/InnerDetector/InDetDetDescr/SCT_ReadoutGeometry/CMakeLists.txt index ddd658b51c20..8134efbe197f 100644 --- a/InnerDetector/InDetDetDescr/SCT_ReadoutGeometry/CMakeLists.txt +++ b/InnerDetector/InDetDetDescr/SCT_ReadoutGeometry/CMakeLists.txt @@ -1,42 +1,17 @@ -################################################################################ -# Package: SCT_ReadoutGeometry -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( SCT_ReadoutGeometry ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaKernel - Control/SGTools - Control/StoreGate - DetectorDescription/AtlasDetDescr - DetectorDescription/GeoPrimitives - DetectorDescription/Identifier - GaudiKernel - InnerDetector/InDetConditions/InDetCondTools - InnerDetector/InDetDetDescr/InDetIdentifier - InnerDetector/InDetDetDescr/InDetReadoutGeometry - Tracking/TrkDetDescr/TrkDetElementBase - Tracking/TrkDetDescr/TrkSurfaces - Tracking/TrkEvent/TrkEventPrimitives - PRIVATE - Database/AthenaPOOL/AthenaPoolUtilities - DetectorDescription/DetDescrCond/DetDescrConditions - DetectorDescription/IdDictDetDescr) - # External dependencies: find_package( CLHEP ) -find_package( Eigen ) find_package( GeoModelCore ) # Component(s) in the package: - atlas_add_library(SCT_ReadoutGeometry - src/*.c* + src/*.cxx PUBLIC_HEADERS SCT_ReadoutGeometry INCLUDE_DIRS ${CLHEP_INCLUDE_DIRS} ${EIGEN_INCLUDE_DIRS} DEFINITIONS ${CLHEP_DEFINITIONS} - LINK_LIBRARIES ${CLHEP_LIBRARIES} ${EIGEN_LIBRARIES} InDetReadoutGeometry AthenaKernel SGTools AtlasDetDescr GeoPrimitives Identifier GaudiKernel InDetIdentifier TrkDetElementBase TrkSurfaces TrkEventPrimitives StoreGateLib SGtests - PRIVATE_LINK_LIBRARIES AthenaPoolUtilities DetDescrConditions IdDictDetDescr) - + LINK_LIBRARIES ${CLHEP_LIBRARIES} ${GEOMODELCORE_LIBRARIES} AthenaKernel GeoPrimitives InDetIdentifier InDetReadoutGeometry TrkSurfaces + PRIVATE_LINK_LIBRARIES AthenaBaseComps AthenaPoolUtilities Identifier StoreGateLib ) diff --git a/InnerDetector/InDetDetDescr/StripGeoModelXml/CMakeLists.txt b/InnerDetector/InDetDetDescr/StripGeoModelXml/CMakeLists.txt index 1178d2a8b088..3ff8dd637737 100644 --- a/InnerDetector/InDetDetDescr/StripGeoModelXml/CMakeLists.txt +++ b/InnerDetector/InDetDetDescr/StripGeoModelXml/CMakeLists.txt @@ -1,50 +1,18 @@ -################################################################################ -# Package: StripGeoModelXml -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( StripGeoModelXml ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaKernel - Database/AthenaPOOL/RDBAccessSvc - DetectorDescription/GeoModelXml - DetectorDescription/GeoModel/GeoModelUtilities - GaudiKernel - InnerDetector/InDetDetDescr/InDetGeoModelUtils - InnerDetector/InDetDetDescr/SCT_ReadoutGeometry - InnerDetector/InDetDetDescr/SCT_ReadoutGeometry - InnerDetector/InDetDetDescr/InDetReadoutGeometry - InnerDetector/InDetSimEvent - PRIVATE - Control/CLIDSvc - Control/SGTools - Control/StoreGate - Database/AthenaPOOL/AthenaPoolUtilities - DetectorDescription/DetDescrCond/DetDescrConditions - DetectorDescription/GeoModel/GeoModelInterfaces - DetectorDescription/GeometryDBSvc - DetectorDescription/Identifier - InnerDetector/InDetDetDescr/InDetIdentifier - Tools/PathResolver ) - -# External dependencies: -find_package( Boost COMPONENTS filesystem thread system ) -find_package( CORAL COMPONENTS CoralBase CoralKernel RelationalAccess ) - # Component(s) in the package: atlas_add_library( StripGeoModelXmlLib src/*.cxx PUBLIC_HEADERS StripGeoModelXml - INCLUDE_DIRS ${Boost_INCLUDE_DIRS} ${CORAL_INCLUDE_DIRS} - LINK_LIBRARIES ${Boost_LIBRARIES} ${CORAL_LIBRARIES} AthenaKernel GeoModelXml GeoModelUtilities GaudiKernel InDetGeoModelUtils InDetReadoutGeometry SCT_ReadoutGeometry SCT_ReadoutGeometry StoreGateLib InDetSimEvent - PRIVATE_LINK_LIBRARIES SGTools AthenaPoolUtilities DetDescrConditions Identifier InDetIdentifier PathResolver) + LINK_LIBRARIES GaudiKernel GeoModelUtilities GeoModelXml InDetGeoModelUtils SCT_ReadoutGeometry + PRIVATE_LINK_LIBRARIES ${GEOMODELCORE_LIBRARIES} AthenaPoolUtilities DetDescrConditions GeoModelInterfaces GeometryDBSvcLib InDetReadoutGeometry InDetSimEvent PathResolver RDBAccessSvcLib SGTools StoreGateLib ) atlas_add_component( StripGeoModelXml src/components/*.cxx - INCLUDE_DIRS ${Boost_INCLUDE_DIRS} ${CORAL_INCLUDE_DIRS} ${CLHEP_INCLUDE_DIRS} - LINK_LIBRARIES ${Boost_LIBRARIES} ${CORAL_LIBRARIES} ${CLHEP_LIBRARIES} AthenaKernel GeoModelXml GeoModelUtilities GaudiKernel InDetGeoModelUtils InDetReadoutGeometry SCT_ReadoutGeometry SCT_ReadoutGeometry SGTools StoreGateLib SGtests AthenaPoolUtilities DetDescrConditions Identifier InDetIdentifier StripGeoModelXmlLib ) + LINK_LIBRARIES StripGeoModelXmlLib ) # Install files from the package: -atlas_install_python_modules( python/*.py ) +atlas_install_python_modules( python/*.py POST_BUILD_CMD ${ATLAS_FLAKE8} ) diff --git a/InnerDetector/InDetDetDescr/TRT_Cabling/CMakeLists.txt b/InnerDetector/InDetDetDescr/TRT_Cabling/CMakeLists.txt index d40a987d849b..6ae8073af801 100644 --- a/InnerDetector/InDetDetDescr/TRT_Cabling/CMakeLists.txt +++ b/InnerDetector/InDetDetDescr/TRT_Cabling/CMakeLists.txt @@ -1,22 +1,8 @@ -################################################################################ -# Package: TRT_Cabling -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( TRT_Cabling ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - GaudiKernel - PRIVATE - Control/AthenaBaseComps - Control/CxxUtils - DetectorDescription/Identifier - InnerDetector/InDetDetDescr/InDetIdentifier - InnerDetector/InDetDetDescr/InDetReadoutGeometry - InnerDetector/InDetDetDescr/TRT_ReadoutGeometry - Tools/PathResolver ) - # External dependencies: find_package( tdaq-common ) @@ -32,7 +18,7 @@ atlas_add_component( TRT_Cabling src/*.cxx src/components/*.cxx INCLUDE_DIRS ${TDAQ-COMMON_INCLUDE_DIRS} - LINK_LIBRARIES ${TDAQ-COMMON_LIBRARIES} GaudiKernel AthenaBaseComps CxxUtils Identifier InDetIdentifier InDetReadoutGeometry TRT_ReadoutGeometry PathResolver TRT_CablingLib ) + LINK_LIBRARIES ${TDAQ-COMMON_LIBRARIES} GaudiKernel AthenaBaseComps CxxUtils Identifier InDetIdentifier TRT_ReadoutGeometry PathResolver TRT_CablingLib ) # Install files from the package: atlas_install_runtime( share/*.dat ) diff --git a/InnerDetector/InDetDetDescr/TRT_GeoModel/CMakeLists.txt b/InnerDetector/InDetDetDescr/TRT_GeoModel/CMakeLists.txt index 35003c58ac51..7d9342d5b101 100644 --- a/InnerDetector/InDetDetDescr/TRT_GeoModel/CMakeLists.txt +++ b/InnerDetector/InDetDetDescr/TRT_GeoModel/CMakeLists.txt @@ -1,50 +1,22 @@ -################################################################################ -# Package: TRT_GeoModel -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( TRT_GeoModel ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaKernel - Database/RDBAccessSvc - DetectorDescription/GeoModel/GeoModelUtilities - GaudiKernel - InnerDetector/InDetConditions/TRT_ConditionsServices - InnerDetector/InDetDetDescr/InDetGeoModelUtils - InnerDetector/InDetDetDescr/InDetReadoutGeometry - InnerDetector/InDetDetDescr/TRT_ReadoutGeometry - PRIVATE - Control/CxxUtils - Control/SGTools - Control/StoreGate - Database/AthenaPOOL/AthenaPoolUtilities - DetectorDescription/DetDescrCond/DetDescrConditions - DetectorDescription/GeoModel/GeoModelInterfaces - DetectorDescription/GeoPrimitives - DetectorDescription/GeometryDBSvc - DetectorDescription/IdDictDetDescr - InnerDetector/InDetConditions/TRT_ConditionsData - InnerDetector/InDetDetDescr/InDetIdentifier ) - # External dependencies: -find_package( CORAL COMPONENTS CoralBase CoralKernel RelationalAccess ) -find_package( Eigen ) find_package( GeoModelCore ) # Component(s) in the package: atlas_add_component( TRT_GeoModel - src/*.cxx + src/*.cxx src/components/*.cxx - INCLUDE_DIRS ${CORAL_INCLUDE_DIRS} - LINK_LIBRARIES ${CORAL_LIBRARIES} ${GEOMODELCORE_LIBRARIES} AthenaKernel GeoModelUtilities GaudiKernel TRT_ConditionsServicesLib InDetGeoModelUtils InDetReadoutGeometry TRT_ReadoutGeometry CxxUtils SGTools StoreGateLib SGtests AthenaPoolUtilities DetDescrConditions IdDictDetDescr TRT_ConditionsData InDetIdentifier GeometryDBSvcLib ) - -atlas_add_test( TRT_GMConfig_test - SCRIPT test/TRT_GMConfig_test.py - PROPERTIES TIMEOUT 600 ) + LINK_LIBRARIES ${GEOMODELCORE_LIBRARIES} AthenaKernel GeoModelInterfaces GeoModelUtilities GeoPrimitives GaudiKernel TRT_ConditionsServicesLib InDetGeoModelUtils InDetReadoutGeometry TRT_ReadoutGeometry CxxUtils SGTools StoreGateLib AthenaPoolUtilities DetDescrConditions IdDictDetDescr TRT_ConditionsData InDetIdentifier GeometryDBSvcLib RDBAccessSvcLib ) # Install files from the package: -atlas_install_python_modules( python/*.py ) +atlas_install_python_modules( python/*.py POST_BUILD_CMD ${ATLAS_FLAKE8} ) atlas_install_scripts( test/*.py ) +# Test(s) in the package: +atlas_add_test( TRT_GMConfig_test + SCRIPT test/TRT_GMConfig_test.py + PROPERTIES TIMEOUT 600 ) diff --git a/InnerDetector/InDetDetDescr/TRT_ReadoutGeometry/CMakeLists.txt b/InnerDetector/InDetDetDescr/TRT_ReadoutGeometry/CMakeLists.txt index 4c878c7efe7b..17b92b402c59 100644 --- a/InnerDetector/InDetDetDescr/TRT_ReadoutGeometry/CMakeLists.txt +++ b/InnerDetector/InDetDetDescr/TRT_ReadoutGeometry/CMakeLists.txt @@ -1,41 +1,16 @@ -################################################################################ -# Package: TRT_ReadoutGeometry -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( TRT_ReadoutGeometry ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaKernel - Control/SGTools - Control/StoreGate - DetectorDescription/AtlasDetDescr - DetectorDescription/GeoPrimitives - DetectorDescription/Identifier - GaudiKernel - InnerDetector/InDetDetDescr/InDetIdentifier - InnerDetector/InDetDetDescr/InDetReadoutGeometry - Tracking/TrkDetDescr/TrkDetElementBase - Tracking/TrkDetDescr/TrkSurfaces - Tracking/TrkEvent/TrkEventPrimitives - DetectorDescription/DetDescrCond/DetDescrConditions - PRIVATE - Database/AthenaPOOL/AthenaPoolUtilities - DetectorDescription/IdDictDetDescr - InnerDetector/InDetConditions/TRT_ConditionsData ) - # External dependencies: find_package( CLHEP ) -find_package( Eigen ) # Component(s) in the package: - atlas_add_library( TRT_ReadoutGeometry - src/*.c* + src/*.cxx PUBLIC_HEADERS TRT_ReadoutGeometry - INCLUDE_DIRS ${CLHEP_INCLUDE_DIRS} ${EIGEN_INCLUDE_DIRS} + INCLUDE_DIRS ${CLHEP_INCLUDE_DIRS} DEFINITIONS ${CLHEP_DEFINITIONS} - LINK_LIBRARIES ${CLHEP_LIBRARIES} ${EIGEN_LIBRARIES} InDetReadoutGeometry AthenaKernel SGTools AtlasDetDescr GeoPrimitives Identifier GaudiKernel InDetIdentifier TrkDetElementBase TrkSurfaces TrkEventPrimitives StoreGateLib SGtests - PRIVATE_LINK_LIBRARIES AthenaPoolUtilities DetDescrConditions IdDictDetDescr TRT_ConditionsData ) - + LINK_LIBRARIES ${CLHEP_LIBRARIES} ${GEOMODELCORE_LIBRARIES} AthenaBaseComps AthenaKernel CxxUtils GeoPrimitives Identifier InDetIdentifier InDetReadoutGeometry TrkDetElementBase + PRIVATE_LINK_LIBRARIES DetDescrConditions GaudiKernel GeoModelUtilities StoreGateLib TRT_ConditionsData TrkSurfaces ) -- GitLab From 96b9b2dc147232b6eb239cb14ba116b5587df029 Mon Sep 17 00:00:00 2001 From: Frank Winklmeier <fwinkl@cern> Date: Wed, 21 Oct 2020 17:17:09 +0200 Subject: [PATCH 3/8] InDetConditions: cmake cleanup - remove `atlas_depends_on_subdirs` - update external and link dependencies - do not install component headers - enable flake8 where applicable --- .../BeamSpotConditions/CMakeLists.txt | 9 ++-- .../BeamSpotConditionsData/CMakeLists.txt | 25 +++------- .../InDetBeamSpotService/CMakeLists.txt | 20 +------- .../InDetByteStreamErrors/CMakeLists.txt | 9 ++-- .../CMakeLists.txt | 17 ++----- .../InDetCondFolders/CMakeLists.txt | 7 +-- .../InDetCondTools/CMakeLists.txt | 17 +------ .../InDetConditionsAthenaPool/CMakeLists.txt | 16 +------ .../CMakeLists.txt | 10 +--- .../InDetCoolCoralClientUtils/CMakeLists.txt | 12 ++--- .../PixelConditionsAlgorithms/CMakeLists.txt | 23 ++------- .../PixelConditionsData/CMakeLists.txt | 9 ++-- .../PixelConditionsTools/CMakeLists.txt | 41 ++-------------- .../SCT_ConditionsAlgorithms/CMakeLists.txt | 27 +---------- .../SCT_ConditionsData/CMakeLists.txt | 18 ++----- .../SCT_ConditionsTools/CMakeLists.txt | 48 +++++-------------- .../SiLorentzAngleTool/CMakeLists.txt | 36 ++------------ .../SiPropertiesTool/CMakeLists.txt | 28 ++--------- .../TRT_ConditionsAlgs/CMakeLists.txt | 23 +-------- .../TRT_ConditionsData/CMakeLists.txt | 27 ++--------- .../TRT_ConditionsServices/CMakeLists.txt | 48 +++---------------- .../TRT_RecoConditionsServices/CMakeLists.txt | 23 +-------- 22 files changed, 85 insertions(+), 408 deletions(-) diff --git a/InnerDetector/InDetConditions/BeamSpotConditions/CMakeLists.txt b/InnerDetector/InDetConditions/BeamSpotConditions/CMakeLists.txt index 7e52e267694a..deeef4699b2f 100644 --- a/InnerDetector/InDetConditions/BeamSpotConditions/CMakeLists.txt +++ b/InnerDetector/InDetConditions/BeamSpotConditions/CMakeLists.txt @@ -3,12 +3,15 @@ # Declare the package name. atlas_subdir( BeamSpotConditions ) +# External dependencies: +find_package( CORAL COMPONENTS CoralBase ) + # Component(s) in the package. atlas_add_component( BeamSpotConditions src/*.h src/*.cxx src/components/*.cxx - LINK_LIBRARIES GaudiKernel AthenaPoolUtilities AthenaBaseComps AthenaKernel + INCLUDE_DIRS ${CORAL_INCLUDE_DIRS} + LINK_LIBRARIES ${CORAL_LIBRARIES} GaudiKernel AthenaPoolUtilities AthenaBaseComps StoreGateLib BeamSpotConditionsData ) # Install files from the package: -atlas_install_python_modules( python/*.py - POST_BUILD_CMD ${ATLAS_FLAKE8} ) +atlas_install_python_modules( python/*.py POST_BUILD_CMD ${ATLAS_FLAKE8} ) diff --git a/InnerDetector/InDetConditions/BeamSpotConditionsData/CMakeLists.txt b/InnerDetector/InDetConditions/BeamSpotConditionsData/CMakeLists.txt index cbcc424e0189..184134a8b8e6 100644 --- a/InnerDetector/InDetConditions/BeamSpotConditionsData/CMakeLists.txt +++ b/InnerDetector/InDetConditions/BeamSpotConditionsData/CMakeLists.txt @@ -1,29 +1,16 @@ -################################# -# Package: BeamSpotConditionsData -################################# +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration -# package name +# Declare the package name: atlas_subdir( BeamSpotConditionsData ) # possible extra dependencies -set ( extra_dep ) set ( extra_lib ) if ( NOT SIMULATIONBASE ) - set ( extra_dep Tracking/TrkEvent/VxVertex ) set ( extra_lib VxVertex ) endif() -# dependencies -atlas_depends_on_subdirs( PUBLIC - Control/AthenaKernel - Database/AthenaPOOL/AthenaPoolUtilities - DetectorDescription/GeoPrimitives - ${extra_dep} - ) - -# components +# Component(s) in the package: atlas_add_library( BeamSpotConditionsData - src/*.cxx - PUBLIC_HEADERS BeamSpotConditionsData - LINK_LIBRARIES AthenaPoolUtilities GeoPrimitives ${extra_lib} - ) + src/*.cxx + PUBLIC_HEADERS BeamSpotConditionsData + LINK_LIBRARIES AthenaKernel GeoPrimitives ${extra_lib} ) diff --git a/InnerDetector/InDetConditions/InDetBeamSpotService/CMakeLists.txt b/InnerDetector/InDetConditions/InDetBeamSpotService/CMakeLists.txt index 1a35e84ad97e..1d04e8101e1f 100644 --- a/InnerDetector/InDetConditions/InDetBeamSpotService/CMakeLists.txt +++ b/InnerDetector/InDetConditions/InDetBeamSpotService/CMakeLists.txt @@ -1,32 +1,14 @@ -# $Id: CMakeLists.txt 773822 2016-09-19 10:04:57Z krasznaa $ -################################################################################ -# Package: InDetBeamSpotService -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( InDetBeamSpotService ) # Possible extra dependencies: -set( extra_dep ) set( extra_lib ) if( NOT SIMULATIONBASE ) - set( extra_dep Tracking/TrkEvent/VxVertex ) set( extra_lib VxVertex ) endif() -# Declare the package's dependencies: -atlas_depends_on_subdirs( - PUBLIC - DetectorDescription/GeoPrimitives - GaudiKernel - ${extra_dep} - PRIVATE - Control/AthenaBaseComps - Control/CxxUtils - Control/StoreGate - Database/AthenaPOOL/AthenaPoolUtilities - Event/EventPrimitives ) - # External dependencies: find_package( CORAL COMPONENTS CoralBase ) diff --git a/InnerDetector/InDetConditions/InDetByteStreamErrors/CMakeLists.txt b/InnerDetector/InDetConditions/InDetByteStreamErrors/CMakeLists.txt index 4c252f00bcb9..32c5bce4d005 100644 --- a/InnerDetector/InDetConditions/InDetByteStreamErrors/CMakeLists.txt +++ b/InnerDetector/InDetConditions/InDetByteStreamErrors/CMakeLists.txt @@ -1,6 +1,4 @@ -################################################################################ -# Package: InDetByteStreamErrors -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( InDetByteStreamErrors ) @@ -9,10 +7,9 @@ atlas_subdir( InDetByteStreamErrors ) atlas_add_dictionary( InDetByteStreamErrorsDict InDetByteStreamErrors/InDetByteStreamErrorsDict.h InDetByteStreamErrors/selection.xml - LINK_LIBRARIES AthContainers Identifier ) + LINK_LIBRARIES AthContainers Identifier InDetByteStreamErrors ) atlas_add_library( InDetByteStreamErrors src/*.cxx PUBLIC_HEADERS InDetByteStreamErrors - LINK_LIBRARIES AthenaKernel EventContainers ) - + LINK_LIBRARIES AthContainers AthenaKernel EventContainers Identifier ) diff --git a/InnerDetector/InDetConditions/InDetByteStreamErrorsAthenaPool/CMakeLists.txt b/InnerDetector/InDetConditions/InDetByteStreamErrorsAthenaPool/CMakeLists.txt index 7a30a784da66..e6d921e9ccc6 100644 --- a/InnerDetector/InDetConditions/InDetByteStreamErrorsAthenaPool/CMakeLists.txt +++ b/InnerDetector/InDetConditions/InDetByteStreamErrorsAthenaPool/CMakeLists.txt @@ -1,6 +1,4 @@ -################################################################################ -# Package: InDetByteStreamErrorsAthenaPool -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( InDetByteStreamErrorsAthenaPool ) @@ -11,25 +9,23 @@ atlas_add_poolcnv_library( InDetByteStreamErrorsAthenaPoolPoolCnv src/*.cxx InDetByteStreamErrors/TRT_BSIdErrContainer.h InDetByteStreamErrors/TRT_BSErrContainer.h InDetByteStreamErrors/IDCInDetBSErrContainer.h - LINK_LIBRARIES AthenaPoolUtilities Identifier InDetByteStreamErrors TestTools AthAllocators - AthenaPoolCnvSvcLib ) + LINK_LIBRARIES Identifier InDetByteStreamErrors AthAllocators AthenaPoolCnvSvcLib ) atlas_add_dictionary( InDetByteStreamErrorsAthenaPoolCnvDict InDetByteStreamErrorsAthenaPool/InDetByteStreamErrorsAthenaPoolCnvDict.h InDetByteStreamErrorsAthenaPool/selection.xml - LINK_LIBRARIES AthenaPoolUtilities Identifier TestTools AthAllocators - AthenaPoolCnvSvcLib ) + LINK_LIBRARIES Identifier ) # Tests in the package: atlas_add_test( InDetBSErrContainerCnv_p1_test SOURCES test/InDetBSErrContainerCnv_p1_test.cxx src/InDetBSErrContainerCnv_p1.cxx - LINK_LIBRARIES GaudiKernel AthenaPoolCnvSvcLib Identifier AthAllocators InDetByteStreamErrors ) + LINK_LIBRARIES GaudiKernel AthenaPoolCnvSvcLib Identifier InDetByteStreamErrors ) atlas_add_test( IDCInDetBSErrContainerCnv_p1_test SOURCES test/IDCInDetBSErrContainerCnv_p1_test.cxx src/IDCInDetBSErrContainerCnv_p1.cxx - LINK_LIBRARIES GaudiKernel AthenaPoolCnvSvcLib Identifier AthAllocators InDetByteStreamErrors ) + LINK_LIBRARIES GaudiKernel AthenaPoolCnvSvcLib Identifier InDetByteStreamErrors ) #atlas_add_test( TRT_BSErrContainerCnv_p1_test # SOURCES test/TRT_BSErrContainerCnv_p1_test.cxx @@ -40,6 +36,3 @@ atlas_add_test( TRT_BSIdErrContainerCnv_p1_test SOURCES test/TRT_BSIdErrContainerCnv_p1_test.cxx src/TRT_BSIdErrContainerCnv_p1.cxx LINK_LIBRARIES GaudiKernel AthenaPoolCnvSvcLib Identifier AthAllocators InDetByteStreamErrors ) - -# Install files from the package: -atlas_install_headers( InDetByteStreamErrorsAthenaPool ) diff --git a/InnerDetector/InDetConditions/InDetCondFolders/CMakeLists.txt b/InnerDetector/InDetConditions/InDetCondFolders/CMakeLists.txt index 66ea19c6c02b..e78fd7bf8749 100644 --- a/InnerDetector/InDetConditions/InDetCondFolders/CMakeLists.txt +++ b/InnerDetector/InDetConditions/InDetCondFolders/CMakeLists.txt @@ -1,10 +1,7 @@ -################################################################################ -# Package: InDetCondFolders -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( InDetCondFolders ) # Install files from the package: -atlas_install_python_modules( python/*.py ) - +atlas_install_python_modules( python/*.py POST_BUILD_CMD ${ATLAS_FLAKE8} ) diff --git a/InnerDetector/InDetConditions/InDetCondTools/CMakeLists.txt b/InnerDetector/InDetConditions/InDetCondTools/CMakeLists.txt index 958505d88648..40f38a5e422f 100644 --- a/InnerDetector/InDetConditions/InDetCondTools/CMakeLists.txt +++ b/InnerDetector/InDetConditions/InDetCondTools/CMakeLists.txt @@ -1,22 +1,9 @@ -################################################################################ -# Package: InDetCondTools -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( InDetCondTools ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaKernel - DetectorDescription/GeoPrimitives - GaudiKernel ) - -# External dependencies: -find_package( Eigen ) - # Component(s) in the package: atlas_add_library( InDetCondTools PUBLIC_HEADERS InDetCondTools - INCLUDE_DIRS ${EIGEN_INCLUDE_DIRS} - LINK_LIBRARIES ${EIGEN_LIBRARIES} AthenaKernel GeoPrimitives GaudiKernel ) - + LINK_LIBRARIES GeoPrimitives GaudiKernel ) diff --git a/InnerDetector/InDetConditions/InDetConditionsAthenaPool/CMakeLists.txt b/InnerDetector/InDetConditions/InDetConditionsAthenaPool/CMakeLists.txt index a429640dec3f..f76ac53816e4 100644 --- a/InnerDetector/InDetConditions/InDetConditionsAthenaPool/CMakeLists.txt +++ b/InnerDetector/InDetConditions/InDetConditionsAthenaPool/CMakeLists.txt @@ -1,19 +1,8 @@ -################################################################################ -# Package: InDetConditionsAthenaPool -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( InDetConditionsAthenaPool ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( - PUBLIC - Database/AthenaPOOL/AthenaPoolUtilities - PRIVATE - GaudiKernel - Database/AthenaPOOL/AthenaPoolCnvSvc - InnerDetector/InDetConditions/TRT_ConditionsData ) - # Component(s) in the package: atlas_add_poolcnv_library( InDetConditionsAthenaPoolPoolCnv src/*.cxx FILES TRT_ConditionsData/TRTStrawStatusData.h @@ -43,6 +32,3 @@ atlas_add_dictionary( InDetConditionsAthenaPoolCnvDict InDetConditionsAthenaPool/selection.xml LINK_LIBRARIES AthenaPoolUtilities AthenaPoolCnvSvcLib TRT_ConditionsData ) - -# Install files from the package: -atlas_install_headers( InDetConditionsAthenaPool ) diff --git a/InnerDetector/InDetConditions/InDetConditionsSummaryService/CMakeLists.txt b/InnerDetector/InDetConditions/InDetConditionsSummaryService/CMakeLists.txt index b36c6e50bb99..e287e68ebd29 100644 --- a/InnerDetector/InDetConditions/InDetConditionsSummaryService/CMakeLists.txt +++ b/InnerDetector/InDetConditions/InDetConditionsSummaryService/CMakeLists.txt @@ -1,15 +1,9 @@ -################################################################################ -# Package: InDetConditionsSummaryService -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( InDetConditionsSummaryService ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - GaudiKernel ) - atlas_add_library( InDetConditionsSummaryService INTERFACE PUBLIC_HEADERS InDetConditionsSummaryService - LINK_LIBRARIES GaudiKernel ) + LINK_LIBRARIES GaudiKernel Identifier ) diff --git a/InnerDetector/InDetConditions/InDetCoolCoralClientUtils/CMakeLists.txt b/InnerDetector/InDetConditions/InDetCoolCoralClientUtils/CMakeLists.txt index fa8895834fc1..89d34c17a117 100644 --- a/InnerDetector/InDetConditions/InDetCoolCoralClientUtils/CMakeLists.txt +++ b/InnerDetector/InDetConditions/InDetCoolCoralClientUtils/CMakeLists.txt @@ -1,6 +1,4 @@ -################################################################################ -# Package: InDetCoolCoralClientUtils -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( InDetCoolCoralClientUtils ) @@ -8,12 +6,12 @@ atlas_subdir( InDetCoolCoralClientUtils ) # External dependencies: find_package( COOL COMPONENTS CoolKernel ) find_package( CORAL COMPONENTS CoralBase CoralKernel RelationalAccess ) -find_package( ROOT COMPONENTS Core Tree MathCore Hist RIO pthread ) # Component(s) in the package: atlas_add_library( InDetCoolCoralClientUtils src/*.cpp PUBLIC_HEADERS InDetCoolCoralClientUtils - INCLUDE_DIRS ${ROOT_INCLUDE_DIRS} ${COOL_INCLUDE_DIRS} ${CORAL_INCLUDE_DIRS} - LINK_LIBRARIES ${ROOT_LIBRARIES} ${COOL_LIBRARIES} ${CORAL_LIBRARIES} CxxUtils ) - + INCLUDE_DIRS ${COOL_INCLUDE_DIRS} + PRIVATE_INCLUDE_DIRS ${CORAL_INCLUDE_DIRS} + LINK_LIBRARIES ${COOL_LIBRARIES} CxxUtils + PRIVATE_LINK_LIBRARIES ${CORAL_LIBRARIES} ) diff --git a/InnerDetector/InDetConditions/PixelConditionsAlgorithms/CMakeLists.txt b/InnerDetector/InDetConditions/PixelConditionsAlgorithms/CMakeLists.txt index ee236f632a43..7ae63b037268 100644 --- a/InnerDetector/InDetConditions/PixelConditionsAlgorithms/CMakeLists.txt +++ b/InnerDetector/InDetConditions/PixelConditionsAlgorithms/CMakeLists.txt @@ -1,30 +1,17 @@ -################################################################################ -# Package: PixelConditionsAlgorithms -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( PixelConditionsAlgorithms ) # External dependencies: find_package( CLHEP ) -find_package( COOL COMPONENTS CoolKernel ) +find_package( CORAL COMPONENTS CoralBase ) # Component(s) in the package: -atlas_add_library( PixelConditionsAlgorithmsLib - PixelConditionsAlgorithms/*.h - INTERFACE - NO_PUBLIC_HEADERS - LINK_LIBRARIES GaudiKernel AthenaKernel AthenaBaseComps PixelConditionsData DetDescrConditions - AthenaPoolUtilities PathResolver TrkGeometry) - atlas_add_component( PixelConditionsAlgorithms src/*.h src/*.cxx src/components/*.cxx - ${COOL_INCLUDE_DIRS} ${CLHEP_INCLUDE_DIRS} - LINK_LIBRARIES ${COOL_LIBRARIES} ${CLHEP_LIBRARIES} CommissionEvent AthenaBaseComps AthenaKernel AthenaPoolUtilities - GaudiKernel PixelConditionsData SGTools StoreGateLib - GeoModelUtilities Identifier InDetIdentifier InDetReadoutGeometry PixelReadoutGeometry - PixelCablingLib PixelConditionsAlgorithmsLib PathResolver ) + INCLUDE_DIRS ${CLHEP_INCLUDE_DIRS} ${CORAL_INCLUDE_DIRS} + LINK_LIBRARIES ${CLHEP_LIBRARIES} ${CORAL_LIBRARIES} AthenaBaseComps AthenaKernel AthenaPoolUtilities CommissionEvent DetDescrConditions GaudiKernel GeoModelUtilities GeoPrimitives Identifier InDetCondTools InDetIdentifier InDetReadoutGeometry PathResolver PixelConditionsData PixelReadoutGeometry StoreGateLib TrkGeometry TrkSurfaces ) # Install files from the package: -atlas_install_python_modules( python/*.py ) - +atlas_install_python_modules( python/*.py POST_BUILD_CMD ${ATLAS_FLAKE8} ) diff --git a/InnerDetector/InDetConditions/PixelConditionsData/CMakeLists.txt b/InnerDetector/InDetConditions/PixelConditionsData/CMakeLists.txt index e1d3edc2ab58..31096f9226da 100644 --- a/InnerDetector/InDetConditions/PixelConditionsData/CMakeLists.txt +++ b/InnerDetector/InDetConditions/PixelConditionsData/CMakeLists.txt @@ -1,6 +1,4 @@ -################################################################################ -# Package: PixelConditionsData -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( PixelConditionsData ) @@ -15,6 +13,5 @@ atlas_add_library( PixelConditionsData PUBLIC_HEADERS PixelConditionsData PRIVATE_INCLUDE_DIRS ${CLHEP_INCLUDE_DIRS} ${ROOT_INCLUDE_DIRS} PRIVATE_DEFINITIONS ${CLHEP_DEFINITIONS} - LINK_LIBRARIES CxxUtils Identifier GeoPrimitives InDetByteStreamErrors - PRIVATE_LINK_LIBRARIES ${CLHEP_LIBRARIES} ${ROOT_LIBRARIES} AthenaKernel AthenaPoolUtilities GaudiKernel ) - + LINK_LIBRARIES AthenaKernel AthenaPoolUtilities GeoPrimitives Identifier InDetByteStreamErrors + PRIVATE_LINK_LIBRARIES ${CLHEP_LIBRARIES} ${ROOT_LIBRARIES} ) diff --git a/InnerDetector/InDetConditions/PixelConditionsTools/CMakeLists.txt b/InnerDetector/InDetConditions/PixelConditionsTools/CMakeLists.txt index bd396954966d..b76593f2e4a1 100644 --- a/InnerDetector/InDetConditions/PixelConditionsTools/CMakeLists.txt +++ b/InnerDetector/InDetConditions/PixelConditionsTools/CMakeLists.txt @@ -1,55 +1,22 @@ -################################################################################ -# Package: PixelConditionsTools -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( PixelConditionsTools ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaKernel - DetectorDescription/GeoPrimitives - GaudiKernel - InnerDetector/InDetConditions/InDetConditionsSummaryService - PRIVATE - Control/AthenaBaseComps - Control/CxxUtils - Control/SGTools - Database/AthenaPOOL/AthenaPoolUtilities - Database/RegistrationServices - DetectorDescription/DetDescrCond/DetDescrConditions - DetectorDescription/Identifier - DetectorDescription/GeoModel/GeoModelInterfaces - DetectorDescription/GeoModel/GeoModelUtilities - InnerDetector/InDetConditions/PixelConditionsData - InnerDetector/InDetDetDescr/InDetIdentifier - InnerDetector/InDetDetDescr/InDetReadoutGeometry - InnerDetector/InDetDetDescr/PixelCabling - InnerDetector/InDetDetDescr/PixelGeoModel - InnerDetector/InDetConditions/InDetByteStreamErrors - Tools/PathResolver ) - -# External dependencies: -find_package( CLHEP ) -find_package( Eigen ) - atlas_add_library( PixelConditionsToolsLib PixelConditionsTools/*.h INTERFACE PUBLIC_HEADERS PixelConditionsTools - LINK_LIBRARIES GaudiKernel AthenaPoolUtilities PixelConditionsData InDetConditionsSummaryService ${interface_extra_lib} ) + LINK_LIBRARIES GaudiKernel InDetConditionsSummaryService ) # Component(s) in the package: atlas_add_component( PixelConditionsTools src/*.cxx src/components/*.cxx - INCLUDE_DIRS ${CLHEP_INCLUDE_DIRS} ${EIGEN_INCLUDE_DIRS} - LINK_LIBRARIES ${CLHEP_LIBRARIES} ${EIGEN_LIBRARIES} PixelConditionsToolsLib AthenaKernel GeoPrimitives GaudiKernel AthenaBaseComps - CxxUtils SGTools AthenaPoolUtilities DetDescrConditions Identifier PixelCablingLib PixelConditionsData InDetIdentifier - GeoModelUtilities InDetReadoutGeometry PathResolver PixelGeoModelLib InDetByteStreamErrors ) + LINK_LIBRARIES AthenaBaseComps AthenaKernel CxxUtils GaudiKernel Identifier InDetByteStreamErrors InDetConditionsSummaryService InDetIdentifier PixelCablingLib PixelConditionsData PixelConditionsToolsLib StoreGateLib ) # Install files from the package: atlas_install_joboptions( share/*.py ) -atlas_install_python_modules( python/*.py ) +atlas_install_python_modules( python/*.py POST_BUILD_CMD ${ATLAS_FLAKE8} ) atlas_install_runtime( share/*.txt share/*.py ) diff --git a/InnerDetector/InDetConditions/SCT_ConditionsAlgorithms/CMakeLists.txt b/InnerDetector/InDetConditions/SCT_ConditionsAlgorithms/CMakeLists.txt index 03a58ba39206..ea2bc8b58e8e 100644 --- a/InnerDetector/InDetConditions/SCT_ConditionsAlgorithms/CMakeLists.txt +++ b/InnerDetector/InDetConditions/SCT_ConditionsAlgorithms/CMakeLists.txt @@ -1,31 +1,8 @@ -############################################################################### -# Package: SCT_ConditionsAlgorithms -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( SCT_ConditionsAlgorithms ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaBaseComps - Control/AthenaKernel - Control/StoreGate - Database/AthenaPOOL/AthenaPoolUtilities - DetectorDescription/Identifier - DetectorDescription/DetDescrCond/DetDescrConditions - DetectorDescription/GeoModel/GeoModelUtilities - DetectorDescription/GeoPrimitives - GaudiKernel - InnerDetector/InDetConditions/InDetConditionsSummaryService - InnerDetector/InDetConditions/SCT_ConditionsData - InnerDetector/InDetConditions/SCT_ConditionsTools - InnerDetector/InDetDetDescr/SCT_Cabling - PRIVATE - InnerDetector/InDetDetDescr/InDetIdentifier - InnerDetector/InDetDetDescr/InDetReadoutGeometry - InnerDetector/InDetDetDescr/SCT_ReadoutGeometry - Tracking/TrkDetDescr/TrkGeometry ) - # External dependencies: find_package( Boost ) @@ -34,7 +11,7 @@ atlas_add_component( SCT_ConditionsAlgorithms src/*.cxx src/components/*.cxx INCLUDE_DIRS ${Boost_INCLUDE_DIRS} - LINK_LIBRARIES ${Boost_LIBRARIES} AthenaBaseComps AthenaKernel StoreGateLib SGtests Identifier DetDescrConditions GeoModelUtilities GaudiKernel SCT_ConditionsData SCT_CablingLib AthenaPoolUtilities InDetIdentifier InDetReadoutGeometry SCT_ReadoutGeometry TrkGeometry SCT_ConditionsToolsLib ) + LINK_LIBRARIES ${Boost_LIBRARIES} AthenaBaseComps StoreGateLib Identifier DetDescrConditions GeoModelUtilities GeoPrimitives GaudiKernel SCT_ConditionsData SCT_CablingLib AthenaPoolUtilities InDetConditionsSummaryService InDetIdentifier InDetReadoutGeometry SCT_ReadoutGeometry TrkGeometry TrkSurfaces SCT_ConditionsToolsLib ) atlas_add_test( TestCalibChipRead SCRIPT athena.py --threads=5 SCT_ConditionsAlgorithms/testCalibChipRead.py diff --git a/InnerDetector/InDetConditions/SCT_ConditionsData/CMakeLists.txt b/InnerDetector/InDetConditions/SCT_ConditionsData/CMakeLists.txt index 756c60bfff25..a66d908fb9fc 100644 --- a/InnerDetector/InDetConditions/SCT_ConditionsData/CMakeLists.txt +++ b/InnerDetector/InDetConditions/SCT_ConditionsData/CMakeLists.txt @@ -1,24 +1,16 @@ -################################################################################ -# Package: SCT_ConditionsData -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( SCT_ConditionsData ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaBaseComps - Control/AthenaKernel - Database/AthenaPOOL/AthenaPoolUtilities - DetectorDescription/Identifier ) - atlas_add_dictionary( SCT_ConditionsDataDict SCT_ConditionsData/SCT_ConditionsDataDict.h - SCT_ConditionsData/selection.xml ) + SCT_ConditionsData/selection.xml + LINK_LIBRARIES SCT_ConditionsData ) # Component(s) in the package: atlas_add_library( SCT_ConditionsData src/*.cxx PUBLIC_HEADERS SCT_ConditionsData - LINK_LIBRARIES AthenaPoolUtilities Identifier AthenaBaseComps ) - + LINK_LIBRARIES AthenaKernel AthenaPoolUtilities Identifier + PRIVATE_LINK_LIBRARIES AthenaBaseComps ) diff --git a/InnerDetector/InDetConditions/SCT_ConditionsTools/CMakeLists.txt b/InnerDetector/InDetConditions/SCT_ConditionsTools/CMakeLists.txt index 08e859521d21..8b25777dc155 100644 --- a/InnerDetector/InDetConditions/SCT_ConditionsTools/CMakeLists.txt +++ b/InnerDetector/InDetConditions/SCT_ConditionsTools/CMakeLists.txt @@ -1,33 +1,8 @@ -############################################################################### -# Package: SCT_ConditionsTools -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( SCT_ConditionsTools ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaKernel - DetectorDescription/Identifier - GaudiKernel - InnerDetector/InDetConditions/InDetConditionsSummaryService - InnerDetector/InDetConditions/SCT_ConditionsData - InnerDetector/InDetConditions/InDetByteStreamErrors - PRIVATE - Control/AthenaBaseComps - Control/StoreGate - Event/xAOD/xAODEventInfo - Event/EventContainers - Database/RDBAccessSvc - DetectorDescription/GeoModel/GeoModelInterfaces - DetectorDescription/GeoModel/GeoModelUtilities - InnerDetector/InDetConditions/InDetByteStreamErrors - InnerDetector/InDetConditions/SiPropertiesTool - InnerDetector/InDetDetDescr/InDetIdentifier - InnerDetector/InDetDetDescr/InDetReadoutGeometry - InnerDetector/InDetDetDescr/SCT_Cabling - AtlasTest/TestTools ) - # External dependencies: find_package( CLHEP ) find_package( ROOT COMPONENTS Core Tree MathCore Hist RIO pthread ) @@ -35,23 +10,22 @@ find_package( GTest ) find_package( GMock ) # Component(s) in the package: -atlas_add_component ( SCT_ConditionsTools - src/components/*.cxx - INCLUDE_DIRS ${ROOT_INCLUDE_DIRS} ${CLHEP_INCLUDE_DIRS} - LINK_LIBRARIES ${ROOT_LIBRARIES} ${CLHEP_LIBRARIES} AthenaKernel Identifier GeoModelUtilities GaudiKernel AthenaBaseComps StoreGateLib SGtests xAODEventInfo SCT_ConditionsData InDetByteStreamErrors InDetIdentifier InDetReadoutGeometry SCT_CablingLib SiPropertiesToolLib SCT_ConditionsToolsLib ) - - atlas_add_library( SCT_ConditionsToolsLib src/*.cxx PUBLIC_HEADERS SCT_ConditionsTools - INCLUDE_DIRS ${ROOT_INCLUDE_DIRS} ${CLHEP_INCLUDE_DIRS} - LINK_LIBRARIES ${ROOT_LIBRARIES} ${CLHEP_LIBRARIES} AthenaKernel Identifier GeoModelUtilities GaudiKernel AthenaBaseComps StoreGateLib SGtests xAODEventInfo SCT_ConditionsData SiPropertiesToolLib InDetByteStreamErrors InDetIdentifier InDetReadoutGeometry SCT_CablingLib EventContainers InDetConditionsSummaryService RDBAccessSvcLib ) + PRIVATE_INCLUDE_DIRS ${ROOT_INCLUDE_DIRS} ${CLHEP_INCLUDE_DIRS} + LINK_LIBRARIES GaudiKernel GeoModelInterfaces Identifier InDetByteStreamErrors InDetConditionsSummaryService InDetReadoutGeometry RDBAccessSvcLib SCT_CablingLib SCT_ConditionsData + PRIVATE_LINK_LIBRARIES ${CLHEP_LIBRARIES} ${ROOT_LIBRARIES} AthenaBaseComps AthenaKernel GeoModelUtilities InDetIdentifier SiPropertiesToolLib StoreGateLib ) + +atlas_add_component ( SCT_ConditionsTools + src/components/*.cxx + LINK_LIBRARIES SCT_ConditionsToolsLib ) # Add unit tests atlas_add_test( SCT_RODVetoTool_test SOURCES test/SCT_RODVetoTool_test.cxx - INCLUDE_DIRS ${GTEST_INCLUDE_DIRS} ${GMOCK_INCLUDE_DIRS} - LINK_LIBRARIES TestTools SCT_ConditionsToolsLib ${GTEST_LIBRARIES} ${GMOCK_LIBRARIES} + INCLUDE_DIRS ${GTEST_INCLUDE_DIRS} ${GMOCK_INCLUDE_DIRS} + LINK_LIBRARIES ${GTEST_LIBRARIES} ${GMOCK_LIBRARIES} AthenaKernel GaudiKernel InDetIdentifier SCT_ConditionsToolsLib StoreGateLib TestTools ENVIRONMENT "JOBOPTSEARCHPATH=${CMAKE_CURRENT_SOURCE_DIR}/share" ) atlas_add_test( SCT_ConditionsConfig_test @@ -59,6 +33,6 @@ atlas_add_test( SCT_ConditionsConfig_test PROPERTIES TIMEOUT 300 ) # Install files from the package: -atlas_install_python_modules( python/*.py ) +atlas_install_python_modules( python/*.py POST_BUILD_CMD ${ATLAS_FLAKE8} ) atlas_install_scripts( share/*.py ) diff --git a/InnerDetector/InDetConditions/SiLorentzAngleTool/CMakeLists.txt b/InnerDetector/InDetConditions/SiLorentzAngleTool/CMakeLists.txt index db384aa64ddc..e915d6a1b590 100644 --- a/InnerDetector/InDetConditions/SiLorentzAngleTool/CMakeLists.txt +++ b/InnerDetector/InDetConditions/SiLorentzAngleTool/CMakeLists.txt @@ -1,42 +1,13 @@ -################################################################################ -# Package: SiLorentzAngleTool -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( SiLorentzAngleTool ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaBaseComps - Control/AthenaKernel - Control/StoreGate - DetectorDescription/GeoPrimitives - GaudiKernel - InnerDetector/InDetConditions/InDetCondTools - InnerDetector/InDetConditions/SiPropertiesTool - InnerDetector/InDetConditions/PixelConditionsData - InnerDetector/InDetConditions/PixelConditionsTools - InnerDetector/InDetConditions/SCT_ConditionsData - PRIVATE - Database/AthenaPOOL/AthenaPoolUtilities - DetectorDescription/Identifier - InnerDetector/InDetConditions/InDetConditionsSummaryService - InnerDetector/InDetDetDescr/InDetIdentifier - InnerDetector/InDetDetDescr/InDetReadoutGeometry - InnerDetector/InDetDetDescr/PixelReadoutGeometry - MagneticField/MagFieldElements - MagneticField/MagFieldConditions - ) - -# External dependencies: -find_package( Eigen ) - # Component(s) in the package: atlas_add_component( SiLorentzAngleTool src/*.cxx src/components/*.cxx - INCLUDE_DIRS ${EIGEN_INCLUDE_DIRS} - LINK_LIBRARIES ${EIGEN_LIBRARIES} AthenaBaseComps AthenaKernel StoreGateLib SGtests GeoPrimitives GaudiKernel SiPropertiesToolLib AthenaPoolUtilities Identifier InDetIdentifier InDetReadoutGeometry PixelReadoutGeometry MagFieldElements MagFieldConditions InDetConditionsSummaryService ) + LINK_LIBRARIES AthenaBaseComps AthenaKernel StoreGateLib GeoPrimitives GaudiKernel SiPropertiesToolLib AthenaPoolUtilities Identifier InDetCondTools InDetIdentifier InDetReadoutGeometry PixelConditionsData PixelReadoutGeometry MagFieldElements MagFieldConditions InDetConditionsSummaryService SCT_ConditionsData ) # Run tests: atlas_add_test( TestSCTLorentzAngle @@ -53,6 +24,5 @@ atlas_add_test( SiLorentzAngleConfig_test PROPERTIES TIMEOUT 30 ) # Install files from the package: -atlas_install_headers( SiLorentzAngleTool ) -atlas_install_python_modules( python/*.py ) +atlas_install_python_modules( python/*.py POST_BUILD_CMD ${ATLAS_FLAKE8} ) atlas_install_joboptions( share/*.py ) diff --git a/InnerDetector/InDetConditions/SiPropertiesTool/CMakeLists.txt b/InnerDetector/InDetConditions/SiPropertiesTool/CMakeLists.txt index f516555cc721..c237a54eb3b2 100644 --- a/InnerDetector/InDetConditions/SiPropertiesTool/CMakeLists.txt +++ b/InnerDetector/InDetConditions/SiPropertiesTool/CMakeLists.txt @@ -1,25 +1,8 @@ -################################################################################ -# Package: SiPropertiesTool -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( SiPropertiesTool ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaBaseComps - Control/AthenaKernel - GaudiKernel - InnerDetector/InDetDetDescr/InDetReadoutGeometry - PRIVATE - Control/StoreGate - DetectorDescription/Identifier - InnerDetector/InDetConditions/InDetConditionsSummaryService - InnerDetector/InDetDetDescr/InDetIdentifier - InnerDetector/InDetConditions/PixelConditionsData - InnerDetector/InDetConditions/PixelConditionsTools - InnerDetector/InDetConditions/SCT_ConditionsData ) - # External dependencies: find_package( CLHEP ) @@ -29,13 +12,12 @@ atlas_add_library( SiPropertiesToolLib PUBLIC_HEADERS SiPropertiesTool PRIVATE_INCLUDE_DIRS ${CLHEP_INCLUDE_DIRS} PRIVATE_DEFINITIONS ${CLHEP_DEFINITIONS} - LINK_LIBRARIES AthenaBaseComps AthenaKernel GaudiKernel InDetReadoutGeometry PixelConditionsData SCT_ConditionsData StoreGateLib SGtests - PRIVATE_LINK_LIBRARIES ${CLHEP_LIBRARIES} Identifier InDetIdentifier InDetConditionsSummaryService ) + LINK_LIBRARIES AthenaKernel GaudiKernel Identifier InDetConditionsSummaryService InDetReadoutGeometry PixelConditionsData SCT_ConditionsData + PRIVATE_LINK_LIBRARIES ${CLHEP_LIBRARIES} AthenaBaseComps InDetIdentifier StoreGateLib ) atlas_add_component( SiPropertiesTool src/components/*.cxx - INCLUDE_DIRS ${CLHEP_INCLUDE_DIRS} - LINK_LIBRARIES ${CLHEP_LIBRARIES} AthenaBaseComps AthenaKernel GaudiKernel InDetReadoutGeometry PixelConditionsData SCT_ConditionsData StoreGateLib SGtests Identifier InDetIdentifier SiPropertiesToolLib InDetConditionsSummaryService ) + LINK_LIBRARIES SiPropertiesToolLib ) # Run tests: atlas_add_test( TestSCTProperties @@ -52,5 +34,5 @@ atlas_add_test( SiPropertiesConfig_test PROPERTIES TIMEOUT 300 ) # Install files from the package: -atlas_install_python_modules( python/*.py ) +atlas_install_python_modules( python/*.py POST_BUILD_CMD ${ATLAS_FLAKE8} ) atlas_install_joboptions( share/*.py ) diff --git a/InnerDetector/InDetConditions/TRT_ConditionsAlgs/CMakeLists.txt b/InnerDetector/InDetConditions/TRT_ConditionsAlgs/CMakeLists.txt index 50ecadd254e2..eba3d78c7ee8 100644 --- a/InnerDetector/InDetConditions/TRT_ConditionsAlgs/CMakeLists.txt +++ b/InnerDetector/InDetConditions/TRT_ConditionsAlgs/CMakeLists.txt @@ -1,24 +1,8 @@ -################################################################################ -# Package: TRT_ConditionsAlgs -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( TRT_ConditionsAlgs ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaBaseComps - GaudiKernel - InnerDetector/InDetConditions/TRT_ConditionsServices - InnerDetector/InDetConditions/TRT_ConditionsData - PRIVATE - InnerDetector/InDetDetDescr/InDetIdentifier - InnerDetector/InDetDetDescr/InDetReadoutGeometry - InnerDetector/InDetDetDescr/TRT_ReadoutGeometry - Database/AthenaPOOL/AthenaPoolUtilities - DetectorDescription/DetDescrCond/DetDescrConditions - DetectorDescription/GeoModel/GeoModelUtilities ) - # External dependencies: find_package( CORAL COMPONENTS CoralBase CoralKernel RelationalAccess ) @@ -27,12 +11,9 @@ atlas_add_component( TRT_ConditionsAlgs src/*.cxx src/components/*.cxx INCLUDE_DIRS ${CORAL_INCLUDE_DIRS} - LINK_LIBRARIES AthenaBaseComps GaudiKernel TRT_ConditionsServicesLib TRT_ConditionsData InDetIdentifier - InDetReadoutGeometry TRT_ReadoutGeometry AthenaPoolUtilities DetDescrConditions GeoModelUtilities ) - + LINK_LIBRARIES LINK_LIBRARIES ${CORAL_LIBRARIES} AthenaBaseComps AthenaKernel AthenaPoolUtilities DetDescrConditions GaudiKernel GeoModelUtilities GeoPrimitives InDetIdentifier StoreGateLib TRT_ConditionsData TRT_ConditionsServicesLib TRT_ReadoutGeometry ) # Install files from the package: -atlas_install_headers( TRT_ConditionsAlgs ) atlas_install_joboptions( share/*.py ) atlas_install_runtime( share/*.txt share/*.py ) diff --git a/InnerDetector/InDetConditions/TRT_ConditionsData/CMakeLists.txt b/InnerDetector/InDetConditions/TRT_ConditionsData/CMakeLists.txt index 21df872f9dc8..f6aa7939737e 100644 --- a/InnerDetector/InDetConditions/TRT_ConditionsData/CMakeLists.txt +++ b/InnerDetector/InDetConditions/TRT_ConditionsData/CMakeLists.txt @@ -1,36 +1,15 @@ -################################################################################ -# Package: TRT_ConditionsData -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( TRT_ConditionsData ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaKernel - Database/AthenaPOOL/AthenaPoolUtilities - DetectorDescription/DetDescrCond/DetDescrConditions - DetectorDescription/Identifier - GaudiKernel - PRIVATE - Tracking/TrkEvent/TrkEventPrimitives) - - - -# External dependencies: -find_package( ROOT COMPONENTS Core Tree MathCore Hist RIO pthread ) - # Component(s) in the package: atlas_add_library( TRT_ConditionsData src/*.cxx PUBLIC_HEADERS TRT_ConditionsData - PRIVATE_INCLUDE_DIRS ${ROOT_INCLUDE_DIRS} - LINK_LIBRARIES AthenaPoolUtilities DetDescrConditions Identifier GaudiKernel AsgTools TrkEventPrimitives - PRIVATE_LINK_LIBRARIES ${ROOT_LIBRARIES} ) + LINK_LIBRARIES AthenaKernel AthenaPoolUtilities Identifier GaudiKernel TrkEventPrimitives ) atlas_add_dictionary( TRT_ConditionsDataDict TRT_ConditionsData/TRT_ConditionsDataDict.h TRT_ConditionsData/selection.xml - INCLUDE_DIRS ${ROOT_INCLUDE_DIRS} - LINK_LIBRARIES ${ROOT_LIBRARIES} AthenaPoolUtilities DetDescrConditions Identifier GaudiKernel AsgTools TRT_ConditionsData ) - + LINK_LIBRARIES TRT_ConditionsData ) diff --git a/InnerDetector/InDetConditions/TRT_ConditionsServices/CMakeLists.txt b/InnerDetector/InDetConditions/TRT_ConditionsServices/CMakeLists.txt index 7189431e7047..aeba1407ad48 100644 --- a/InnerDetector/InDetConditions/TRT_ConditionsServices/CMakeLists.txt +++ b/InnerDetector/InDetConditions/TRT_ConditionsServices/CMakeLists.txt @@ -1,62 +1,28 @@ -################################################################################ -# Package: TRT_ConditionsServices -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( TRT_ConditionsServices ) # Possible extra dependencies: -set( extra_dep ) set( extra_lib ) if( NOT SIMULATIONBASE ) - set( extra_dep InnerDetector/InDetConditions/InDetByteStreamErrors ) set( extra_lib InDetByteStreamErrors ) endif() -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - Control/AthenaKernel - Control/StoreGate - DetectorDescription/GeoPrimitives - Event/EventPrimitives - GaudiKernel - InnerDetector/InDetConditions/InDetConditionsSummaryService - InnerDetector/InDetConditions/TRT_ConditionsData - PRIVATE - Control/AthenaBaseComps - Control/CxxUtils - Control/StoreGate - Database/AthenaPOOL/AthenaPoolUtilities - Database/RDBAccessSvc - Database/RegistrationServices - DetectorDescription/DetDescrCond/DetDescrConditions - DetectorDescription/GeoModel/GeoModelInterfaces - DetectorDescription/GeoModel/GeoModelUtilities - DetectorDescription/Identifier - Event/xAOD/xAODEventInfo - InnerDetector/InDetConditions/InDetCoolCoralClientUtils - InnerDetector/InDetDetDescr/InDetIdentifier - InnerDetector/InDetDetDescr/InDetReadoutGeometry - InnerDetector/InDetDetDescr/TRT_ReadoutGeometry - Tools/PathResolver - ${extra_dep} ) - # External dependencies: -find_package( COOL COMPONENTS CoolKernel ) -find_package( CORAL COMPONENTS CoralBase CoralKernel RelationalAccess ) find_package( ROOT COMPONENTS RIO Core Tree MathCore Hist pthread ) # Component(s) in the package: atlas_add_library( TRT_ConditionsServicesLib src/*.cxx PUBLIC_HEADERS TRT_ConditionsServices - PRIVATE_INCLUDE_DIRS ${ROOT_INCLUDE_DIRS} ${CORAL_INCLUDE_DIRS} ${COOL_INCLUDE_DIRS} - LINK_LIBRARIES AthenaKernel GeoPrimitives EventPrimitives GaudiKernel TRT_ConditionsData StoreGateLib SGtests InDetConditionsSummaryService - PRIVATE_LINK_LIBRARIES ${ROOT_LIBRARIES} ${CORAL_LIBRARIES} ${COOL_LIBRARIES} AthenaBaseComps CxxUtils AthenaPoolUtilities DetDescrConditions GeoModelUtilities Identifier xAODEventInfo InDetCoolCoralClientUtils InDetIdentifier InDetReadoutGeometry TRT_ReadoutGeometry PathResolver RegistrationServicesLib ${extra_lib} RDBAccessSvcLib ) + PRIVATE_INCLUDE_DIRS ${ROOT_INCLUDE_DIRS} + LINK_LIBRARIES LINK_LIBRARIES AthenaKernel AthenaPoolUtilities DetDescrConditions EventPrimitives GaudiKernel GeoPrimitives InDetConditionsSummaryService InDetIdentifier TRT_ConditionsData + PRIVATE_LINK_LIBRARIES ${ROOT_LIBRARIES} AthenaBaseComps CxxUtils GeoModelInterfaces GeoModelUtilities Identifier xAODEventInfo InDetReadoutGeometry TRT_ReadoutGeometry PathResolver RegistrationServicesLib RDBAccessSvcLib ${extra_lib} ) atlas_add_component( TRT_ConditionsServices src/components/*.cxx - INCLUDE_DIRS ${ROOT_INCLUDE_DIRS} ${CORAL_INCLUDE_DIRS} ${COOL_INCLUDE_DIRS} - LINK_LIBRARIES ${ROOT_LIBRARIES} ${CORAL_LIBRARIES} ${COOL_LIBRARIES} AthenaKernel GeoPrimitives EventPrimitives GaudiKernel TRT_ConditionsData AthenaBaseComps CxxUtils StoreGateLib SGtests AthenaPoolUtilities DetDescrConditions GeoModelUtilities Identifier xAODEventInfo EventInfo InDetCoolCoralClientUtils InDetIdentifier InDetReadoutGeometry TRT_ReadoutGeometry PathResolver TRT_ConditionsServicesLib InDetConditionsSummaryService ${extra_lib} RegistrationServicesLib ) + LINK_LIBRARIES TRT_ConditionsServicesLib ) -atlas_install_python_modules( python/*.py ) +# Install files from the package: +atlas_install_python_modules( python/*.py POST_BUILD_CMD ${ATLAS_FLAKE8} ) diff --git a/InnerDetector/InDetConditions/TRT_RecoConditionsServices/CMakeLists.txt b/InnerDetector/InDetConditions/TRT_RecoConditionsServices/CMakeLists.txt index e63b40e015be..13d81215e34d 100644 --- a/InnerDetector/InDetConditions/TRT_RecoConditionsServices/CMakeLists.txt +++ b/InnerDetector/InDetConditions/TRT_RecoConditionsServices/CMakeLists.txt @@ -1,28 +1,9 @@ -################################################################################ -# Package: TRT_RecoConditionsServices -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( TRT_RecoConditionsServices ) -# Declare the package's dependencies: -atlas_depends_on_subdirs( PUBLIC - GaudiKernel - PRIVATE - Control/AthenaBaseComps - Control/StoreGate - InnerDetector/InDetConditions/TRT_ConditionsServices - InnerDetector/InDetDetDescr/InDetIdentifier - InnerDetector/InDetDetDescr/InDetReadoutGeometry - InnerDetector/InDetDetDescr/TRT_ReadoutGeometry - InnerDetector/InDetRecTools/InDetRecToolInterfaces - Tracking/TrkEvent/TrkParameters ) - -# External dependencies: -find_package( Eigen ) - # Component(s) in the package: atlas_add_component( TRT_RecoConditionsServices src/*.cxx src/components/*.cxx - INCLUDE_DIRS ${EIGEN_INCLUDE_DIRS} - LINK_LIBRARIES ${EIGEN_LIBRARIES} GaudiKernel TrkParameters AthenaBaseComps StoreGateLib SGtests InDetIdentifier InDetReadoutGeometry TRT_ReadoutGeometry TRT_ConditionsServicesLib InDetRecToolInterfaces ) + LINK_LIBRARIES GaudiKernel AthenaBaseComps StoreGateLib InDetIdentifier TRT_ReadoutGeometry TRT_ConditionsServicesLib InDetRecToolInterfaces ) -- GitLab From 74bc5411daf34fbb245c67491ce01a9f602a2e9c Mon Sep 17 00:00:00 2001 From: Frank Winklmeier <fwinkl@cern> Date: Wed, 21 Oct 2020 17:18:37 +0200 Subject: [PATCH 4/8] InDetCondFolders: flake8 fixes --- .../InDetCondFolders/python/InDetAlignFolders.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/InnerDetector/InDetConditions/InDetCondFolders/python/InDetAlignFolders.py b/InnerDetector/InDetConditions/InDetCondFolders/python/InDetAlignFolders.py index c6d415906785..bb6952bc8633 100644 --- a/InnerDetector/InDetConditions/InDetCondFolders/python/InDetAlignFolders.py +++ b/InnerDetector/InDetConditions/InDetCondFolders/python/InDetAlignFolders.py @@ -1,8 +1,7 @@ -# Copyright (C) 2002-2019 CERN for the benefit of the ATLAS collaboration +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration from IOVDbSvc.CondDB import conddb -from AthenaCommon.AthenaCommonFlags import athenaCommonFlags from AthenaCommon.DetFlags import DetFlags from AtlasGeoModel.InDetGMJobProperties import InDetGeometryFlags -- GitLab From 2d9edc51598e1dbb456f5f22283fef7073d338c2 Mon Sep 17 00:00:00 2001 From: Frank Winklmeier <fwinkl@cern> Date: Wed, 21 Oct 2020 17:18:38 +0200 Subject: [PATCH 5/8] SiLorentzAngleTool: flake8 fixes --- .../SiLorentzAngleTool/python/PixelLorentzAngleToolSetup.py | 4 +--- .../SiLorentzAngleTool/python/SCTLorentzAngleToolSetup.py | 4 +++- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/InnerDetector/InDetConditions/SiLorentzAngleTool/python/PixelLorentzAngleToolSetup.py b/InnerDetector/InDetConditions/SiLorentzAngleTool/python/PixelLorentzAngleToolSetup.py index 2ae8b234d0b7..2e4863ac0e01 100644 --- a/InnerDetector/InDetConditions/SiLorentzAngleTool/python/PixelLorentzAngleToolSetup.py +++ b/InnerDetector/InDetConditions/SiLorentzAngleTool/python/PixelLorentzAngleToolSetup.py @@ -1,4 +1,4 @@ -# Copyright (C) 2002-2019 CERN for the benefit of the ATLAS collaboration +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration class PixelLorentzAngleToolSetup: "Class to simplify setup of PixelLorentzAngleTool" @@ -19,7 +19,6 @@ class PixelLorentzAngleToolSetup: if not conddb.folderRequested(tempFolder): conddb.addFolder(dbInstance, tempFolder, className="CondAttrListCollection") - from AthenaCommon.GlobalFlags import globalflags from AthenaCommon.AlgSequence import AthSequencer condSeq = AthSequencer("AthCondSeq") @@ -48,7 +47,6 @@ class PixelLorentzAngleToolSetup: SiPropertiesTool = ToolSvc.PixelSiPropertiesTool, UseMagFieldCache = True, UseMagFieldDcs = (not athenaCommonFlags.isOnline())) - pixelSiLorentzAngleCondAlg = condSeq.PixelSiLorentzAngleCondAlg if not hasattr(ToolSvc, "PixelLorentzAngleTool"): from SiLorentzAngleTool.SiLorentzAngleToolConf import SiLorentzAngleTool diff --git a/InnerDetector/InDetConditions/SiLorentzAngleTool/python/SCTLorentzAngleToolSetup.py b/InnerDetector/InDetConditions/SiLorentzAngleTool/python/SCTLorentzAngleToolSetup.py index 19203820aa06..123cd9f0948a 100644 --- a/InnerDetector/InDetConditions/SiLorentzAngleTool/python/SCTLorentzAngleToolSetup.py +++ b/InnerDetector/InDetConditions/SiLorentzAngleTool/python/SCTLorentzAngleToolSetup.py @@ -1,4 +1,4 @@ -# Copyright (C) 2002-2018 CERN for the benefit of the ATLAS collaboration +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration class SCTLorentzAngleToolSetup: "Class to simplify setup of SCTLorentzAngleTool" @@ -78,4 +78,6 @@ class SCTLorentzAngleToolSetup: def forceUseGeoModel(self) : "Force usage of GeoModel defaults" self.SCT_SiliconConditionsTool.ForceUseGeoModel = True + from AthenaCommon import Logging + msg = Logging.logging.getLogger("SCTLorentzAngleToolSetup") msg.warning("Please set forceUseGeoModel in constructor. Unnecessary service, algorithms, folders are configured") -- GitLab From 80ffdc18853422254e8e1ced75973cb1a213bd31 Mon Sep 17 00:00:00 2001 From: Frank Winklmeier <fwinkl@cern> Date: Wed, 21 Oct 2020 17:18:38 +0200 Subject: [PATCH 6/8] SiPropertiesTool: flake8 fixes --- .../SiPropertiesTool/python/PixelSiPropertiesToolSetup.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/InnerDetector/InDetConditions/SiPropertiesTool/python/PixelSiPropertiesToolSetup.py b/InnerDetector/InDetConditions/SiPropertiesTool/python/PixelSiPropertiesToolSetup.py index d72aca0715c3..e156e23ad419 100644 --- a/InnerDetector/InDetConditions/SiPropertiesTool/python/PixelSiPropertiesToolSetup.py +++ b/InnerDetector/InDetConditions/SiPropertiesTool/python/PixelSiPropertiesToolSetup.py @@ -1,4 +1,4 @@ -# Copyright (C) 2002-2019 CERN for the benefit of the ATLAS collaboration +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration class PixelSiPropertiesToolSetup: "Class to simplify setup of PixelSiPropertiesTool and required conditions algorithm" @@ -44,7 +44,6 @@ class PixelSiPropertiesToolSetup: if not conddb.folderRequested(tempFolder): conddb.addFolder(dbInstance, tempFolder, className="CondAttrListCollection") - from AthenaCommon.GlobalFlags import globalflags from AthenaCommon.AlgSequence import AthSequencer condSeq = AthSequencer("AthCondSeq") @@ -56,7 +55,6 @@ class PixelSiPropertiesToolSetup: from PixelConditionsAlgorithms.PixelConditionsAlgorithmsConf import PixelDCSCondTempAlg condSeq += PixelDCSCondTempAlg(name="PixelDCSCondTempAlg", ReadKey=tempFolder) - from AthenaCommon.AppMgr import ToolSvc if not hasattr(condSeq, "PixelSiPropertiesCondAlg"): from SiPropertiesTool.SiPropertiesToolConf import PixelSiPropertiesCondAlg condSeq += PixelSiPropertiesCondAlg(name="PixelSiPropertiesCondAlg") -- GitLab From 1a70838d5248b64b8b3e4049103ecd3a8c647a0c Mon Sep 17 00:00:00 2001 From: Frank Winklmeier <fwinkl@cern> Date: Thu, 22 Oct 2020 09:04:33 +0200 Subject: [PATCH 7/8] SCT_GeoModel: flake8 fix --- .../InDetDetDescr/SCT_GeoModel/python/SCT_GeoModelConfigDb.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/InnerDetector/InDetDetDescr/SCT_GeoModel/python/SCT_GeoModelConfigDb.py b/InnerDetector/InDetDetDescr/SCT_GeoModel/python/SCT_GeoModelConfigDb.py index fbf6bfea3072..324c8f058dde 100644 --- a/InnerDetector/InDetDetDescr/SCT_GeoModel/python/SCT_GeoModelConfigDb.py +++ b/InnerDetector/InDetDetDescr/SCT_GeoModel/python/SCT_GeoModelConfigDb.py @@ -1,6 +1,6 @@ # Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration -from AthenaCommon.CfgGetter import addAlgorithm,addTool +from AthenaCommon.CfgGetter import addTool # add tools and algorithms to the ConfiguredFactory, see: # https://twiki.cern.ch/twiki/bin/viewauth/AtlasComputing/ConfiguredFactory addTool("SCT_GeoModel.SCT_GeoModelOldConfig.getSCT_DetectorTool","SCT_DetectorTool") -- GitLab From 400dba193f3d9b3e6447b72c81906fe63fc19c1b Mon Sep 17 00:00:00 2001 From: Frank Winklmeier <fwinkl@cern> Date: Thu, 22 Oct 2020 09:06:04 +0200 Subject: [PATCH 8/8] PixelMonitoring: cmake fixes, enable flake8 --- .../PixelMonitoring/CMakeLists.txt | 20 ++++++------------- .../python/PixelAthErrorMonAlgCfg.py | 20 +++++++------------ .../python/PixelAthHitMonAlgCfg.py | 11 +++++----- 3 files changed, 18 insertions(+), 33 deletions(-) diff --git a/InnerDetector/InDetMonitoring/PixelMonitoring/CMakeLists.txt b/InnerDetector/InDetMonitoring/PixelMonitoring/CMakeLists.txt index 144bde139400..c52696659bc4 100644 --- a/InnerDetector/InDetMonitoring/PixelMonitoring/CMakeLists.txt +++ b/InnerDetector/InDetMonitoring/PixelMonitoring/CMakeLists.txt @@ -1,22 +1,14 @@ -################################################################################ -# Package: PixelMonitoring -################################################################################ +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # Declare the package name: atlas_subdir( PixelMonitoring ) -# External dependencies: -find_package( ROOT COMPONENTS Core MathCore Hist ) - # Component(s) in the package: atlas_add_component( PixelMonitoring src/*.h src/*.cxx src/components/*.cxx - INCLUDE_DIRS ${ROOT_INCLUDE_DIRS} - LINK_LIBRARIES ${ROOT_LIBRARIES} AthenaMonitoringLib AtlasDetDescr - GaudiKernel InDetRawData InDetPrepRawData TrkTrack - AthenaPoolUtilities GeoPrimitives xAODEventInfo EventPrimitives InDetIdentifier - InDetReadoutGeometry InDetRIO_OnTrack LWHists TrkParameters TrkSpacePoint - TrkTrackSummary TrkToolInterfaces PixelCablingLib PixelGeoModelLib PathResolver - InDetTrackSelectionToolLib InDetConditionsSummaryService PixelConditionsToolsLib ) + LINK_LIBRARIES AthenaMonitoringKernelLib AthenaMonitoringLib AtlasDetDescr EventPrimitives + InDetConditionsSummaryService InDetIdentifier InDetPrepRawData InDetRIO_OnTrack InDetRawData + InDetReadoutGeometry InDetTrackSelectionToolLib PixelCablingLib PixelConditionsData + TrkMeasurementBase TrkRIO_OnTrack TrkToolInterfaces TrkTrack TrkTrackSummary ) -atlas_install_python_modules( python/*.py ) +atlas_install_python_modules( python/*.py POST_BUILD_CMD ${ATLAS_FLAKE8} ) diff --git a/InnerDetector/InDetMonitoring/PixelMonitoring/python/PixelAthErrorMonAlgCfg.py b/InnerDetector/InDetMonitoring/PixelMonitoring/python/PixelAthErrorMonAlgCfg.py index c778ba94a673..a3a2d5505c40 100644 --- a/InnerDetector/InDetMonitoring/PixelMonitoring/python/PixelAthErrorMonAlgCfg.py +++ b/InnerDetector/InDetMonitoring/PixelMonitoring/python/PixelAthErrorMonAlgCfg.py @@ -7,11 +7,10 @@ @brief Configuration of Pixel Error Monitoring Histograms for Run 3 ''' from PixelMonitoring.PixelAthMonitoringBase import define2DProfHist -from PixelMonitoring.PixelAthMonitoringBase import define1DLayers, defineMapVsLumiLayers +from PixelMonitoring.PixelAthMonitoringBase import defineMapVsLumiLayers from PixelMonitoring.PixelAthMonitoringBase import define1DProfLumiLayers -from PixelMonitoring.PixelAthMonitoringBase import layers, totcuts, xbinsem, xminsem, lumibinsx -from PixelMonitoring.PixelAthMonitoringBase import addOnTrackTxt, fullDressTitle -from PixelMonitoring.PixelAthMonitoringBase import runtext +from PixelMonitoring.PixelAthMonitoringBase import layers +from PixelMonitoring.PixelAthMonitoringBase import fullDressTitle from PixelMonitoring.PixelAthMonitoringBase import errbbinsy, errbminsy, errbbsizy, errtbinsy from PixelMonitoring.PixelAthMonitoringBase import ErrBitLabels, ErrCatRODModLabels, ErrCatLabels, ErrCatRODModLabelsNorm, ErrCatLabelsNorm from PixelMonitoring.PixelAthMonitoringBase import ErrStateLabelsFEI3, ErrStateLabelsFEI4 @@ -23,12 +22,8 @@ def PixelAthErrorMonAlgCfg(helper, alg, **kwargs): alg -- algorithm Configurable object returned from addAlgorithm kwargs -- jo agruments ''' - doOnline = kwargs.get('doOnline', False) - doLumiBlock = kwargs.get('doLumiBlock', False) - path = '/Pixel/Errors/' pathExpert = '/Pixel/ErrorsExpert/' - pathLowStat = '/Pixel/LumiBlock/' errorGroup = helper.addGroup(alg, 'Error') @@ -107,12 +102,11 @@ def PixelAthErrorMonAlgCfg(helper, alg, **kwargs): title = fullDressTitle('IBL ServiceRecord Unweighted', False, ';SR',';# errors/event') varName += ';ServiceRecord_Unweighted_IBL' errorGroup.defineHistogram(varName, - type='TH1F', path=pathExpert, title=title, - xbins=40, xmin=-0.5, xmax=39.5) + type='TH1F', path=pathExpert, title=title, + xbins=40, xmin=-0.5, xmax=39.5) varName = 'ServiceRecord_val' title = fullDressTitle('IBL ServiceRecord Weighted with Payload', False, ';SR',';# errors/event') varName += ';ServiceRecord_Weighted_IBL' errorGroup.defineHistogram(varName, weight='ServiceRecord_wgt', - type='TH1F', path=pathExpert, title=title, - xbins=40, xmin=-0.5, xmax=39.5) - + type='TH1F', path=pathExpert, title=title, + xbins=40, xmin=-0.5, xmax=39.5) diff --git a/InnerDetector/InDetMonitoring/PixelMonitoring/python/PixelAthHitMonAlgCfg.py b/InnerDetector/InDetMonitoring/PixelMonitoring/python/PixelAthHitMonAlgCfg.py index 20d240a20e59..50ada6a7aca4 100644 --- a/InnerDetector/InDetMonitoring/PixelMonitoring/python/PixelAthHitMonAlgCfg.py +++ b/InnerDetector/InDetMonitoring/PixelMonitoring/python/PixelAthHitMonAlgCfg.py @@ -1,5 +1,5 @@ # -# Copyright (C) 2002-2019 CERN for the benefit of the ATLAS collaboration +# Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration # ''' @@ -8,9 +8,8 @@ ''' from PixelMonitoring.PixelAthMonitoringBase import define2DProfHist, definePP0Histos, getLayerGroup -from PixelMonitoring.PixelAthMonitoringBase import define1DLayers from PixelMonitoring.PixelAthMonitoringBase import define1DProfLumiLayers -from PixelMonitoring.PixelAthMonitoringBase import layers, totcuts, xbinsem, xminsem, lumibinsx, bcidbinsx +from PixelMonitoring.PixelAthMonitoringBase import layers, lumibinsx, bcidbinsx from PixelMonitoring.PixelAthMonitoringBase import addOnTrackTxt, addOnTrackToPath, fullDressTitle from PixelMonitoring.PixelAthMonitoringBase import runtext @@ -74,8 +73,8 @@ def PixelAthHitMonAlgCfg(helper, alg, **kwargs): fullvarstr+= ',{0}_{1}'.format(histoname, 'val') fullvarstr+= ';' + histoname + '_{0}'.format(layer) layerGroup.defineHistogram(fullvarstr, - type='TProfile', path=pathGroup, title=title, - xbins=bcidbinsx, xmin=-0.5, xmax=-0.5+bcidbinsx) + type='TProfile', path=pathGroup, title=title, + xbins=bcidbinsx, xmin=-0.5, xmax=-0.5+bcidbinsx) histoGroupName = 'Hit_Occupancy_PP0' title = 'Average per module(FE) hit occupancy reset every 5 min' @@ -109,7 +108,7 @@ def PixelAthHitMonAlgCfg(helper, alg, **kwargs): titleLB = 'Hit Level 1 Accept lowstat {0}'.format(layer) + runtext + ';Level 1 Accept;# hits' fullvarstrLB = varName + ';' + histonameLB layerGroup.defineHistogram(fullvarstrLB, - type='TH1F', path=pathLowStat, title=title, + type='TH1F', path=pathLowStat, title=titleLB, xbins=14, xmin=-1.5, xmax=12.5, duration='lowStat') ### -- GitLab