Build Script Updates, master branch (2021.01.15.)
As @krumnack pointed out, with harmonising the builds of all projects in !39481 (merged), I made the builds of AnalysisBase fail (out of the box) on hosts that do not have access to CVMFS. This was because of some unnecessary logic in the LCG_RELEASE_BASE.sh
and TDAQ_RELEASE_BASE.sh
scripts. Which I now removed.
- The build of AnalysisBase does not use the
LCG_RELEASE_BASE
orTDAQ_RELEASE_BASE
environment variables. But it also doesn't hurt it if they are set during its build. This is why I didn't introduce any logic into the common build script for not setting these up for the standalone project.
While at it, I also added another simplification. The build_externals.sh
scripts could be made just a little simpler, by not setting one of the variables that they were setting before.
Unlike !39481 (merged), these should be pretty safe updates...
Merge request reports
Activity
This merge request affects 7 packages:
- Build
- Projects/AnalysisBase
- Projects/AthAnalysis
- Projects/AthGeneration
- Projects/AthSimulation
- Projects/Athena
- Projects/VP1Light
Affected files list will not be printed in this case
Could not map the following files to a domain:
- Build/AtlasBuildScripts/LCG_RELEASE_BASE.sh
- Build/AtlasBuildScripts/TDAQ_RELEASE_BASE.sh
- Build/AtlasBuildScripts/build_project_externals.sh
Adding @jchapman ,@akraszna ,@krumnack ,@vpascuzz ,@rbianchi as watchers
added Build master review-pending-level-1 labels
CI Result FAILURE (hash 88607202)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 26317]mentioned in merge request !39722 (closed)
added review-approved label and removed review-pending-level-1 label
mentioned in commit e4f47359
added sweep:ignore label
mentioned in merge request !41381 (merged)