feat: Update to Release 24 (!12)
1 unresolved thread
1 unresolved thread
- As Release 22 won't be used for analysis update to use Release 24.
- Update Release 22 files to use Release 24 as Release 22 won't be used for analysis.
- Rename Git submodule columnarprototype to ColumnarPrototype to deal with Release 24 ATLAS CMake requiring exact string matching for package names.
- Update to using gitlab-registry.cern.ch/atlas/athena/analysisbase:24.2.3 for testing in CI.
- Use cvmfs-venv to get valid user controlled virtual environments.
Edited by Matthew Feickert
Merge request reports
Activity
assigned to @feickert
... -- Configuring the build of package: ColumnarPrototype CMake Error at /usr/AnalysisBase/24.2.2/InstallArea/x86_64-centos7-gcc11-opt/cmake/modules/AtlasFunctions.cmake:653 (message): The package name ColumnarPrototype does not match the directory name columnarprototype in columnarprototype. Call Stack (most recent call first): columnarprototype/CMakeLists.txt:5 (atlas_subdir) ...
Not really clear why this is happening. Not sure if @krumnack has thoughts.
It is confusing that this is only happening in Release 24 though (Release 22 is fine), so ATLAS CMake has apparently gotten more restrictive.
Edited by Matthew FeickertLooking at the complaining snippet in
/usr/AnalysisBase/24.2.2/InstallArea/x86_64-centos7-gcc11-opt/cmake/modules/AtlasFunctions.cmake
# Get the source path of the package: atlas_get_package_dir( pkgDir ) # Check if the package name matches the (leaf) directory name: get_filename_component( pkgDirName ${pkgDir} NAME ) if( NOT "${name}" STREQUAL "${pkgDirName}" ) message( SEND_ERROR "The package name ${name} does not match " "the directory name ${pkgDirName} in ${pkgDir}." ) endif()
it is entirely absent in
/usr/AnalysisBase/22.2.113/InstallArea/x86_64-centos7-gcc11-opt/cmake/modules/AtlasFunctions.cmake
# Get the source path of the package: atlas_get_package_dir( pkgDir )
so time to go ask why this was done with exact string matching including casing, which seems unnecessary.
mentioned in commit 73b0725c
mentioned in commit 77519de5
Please register or sign in to reply