Install CalorimeterGeom/data in the CMake build (fixes spack ceSimReco) - #1914
Conversation
crystalPos.txt (added in Mu2e#1908) is opened at runtime via ConfigFileLookupPolicy from geom key calorimeter.diskCrystalFile, but CalorimeterGeom had no install rule for data/ -- a spack/CMake install ships without the file and any calo-geometry job dies: Can't find file "Offline/CalorimeterGeom/data/crystalPos.txt" The existing configure_file line only stages the file into the build tree. Add the install(DIRECTORY data ...) rule every other package shipping runtime data uses (Mu2eG4, TrackerConditions, CaloConditions, CalPatRec, CRVConditions, ...). The scons build reads the source tree and is unaffected. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
☀️ The build tests passed at 932a053.
N.B. These results were obtained from a build of this Pull Request at 932a053 after being merged into the base branch at ed58b12. For more information, please check the job page here. |
rlcee
left a comment
There was a problem hiding this comment.
You want to remove the configure_file a few line above
Per review: the install(DIRECTORY data ...) rule makes the configure_file staging redundant; build-tree runs resolve the file from the source tree via MU2E_SEARCH_PATH, matching how Mu2eG4 and the other data-shipping packages work (install rule only). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Removed the configure_file line in ceb9a3a — the install rule alone matches how Mu2eG4 and the other data-shipping packages handle it. |
|
@FNALbuild run build test |
|
⌛ The following tests have been triggered for ceb9a3a: build (Build queue - API unavailable) |
|
☀️ The build tests passed at ceb9a3a.
N.B. These results were obtained from a build of this Pull Request at ceb9a3a after being merged into the base branch at ed58b12. For more information, please check the job page here. |
Fixes the spack/CMake runtime failure reported this morning:
crystalPos.txt(added in #1908) is opened at runtime viaConfigFileLookupPolicyfrom geom keycalorimeter.diskCrystalFile, so any calo-geometry job (e.g. ceSimReco) needs it onMU2E_SEARCH_PATH.CalorimeterGeom/CMakeLists.txthas no install rule fordata/— theconfigure_fileline only stages the file into the build tree — so a spack/CMake install ships without it. The scons build reads the source tree, which is why CI stayed green.One line, matching every other package that ships runtime data (
Mu2eG4,TrackerConditions,CaloConditions,CalPatRec,CRVConditions,CRVResponse,CaloFilters, …):No behavior change for scons or build-tree runs.
🤖 Generated with Claude Code