Fix for reusing of externals build

Second try to caching the externals build. The improvement compared to !32965 (merged) (reverted in !33206 (merged)) is to include the upstream (Athena) project version in the externals stamp file. That way we are rebuilding (and more importantly reinstalling) the externals in case the upstream version changes. Otherwise it would not be found under the release version (ATLINFR-3018).

Also implement the same caching now for AnalysisBase.

cc @akraszna

This merge request affects 4 packages:

• Projects/AnalysisBase
• Projects/AthGeneration
• Projects/AthSimulation
• Projects/Athena

Adding @jchapman ,@akraszna ,@krumnack ,@vpascuzz ,@rbianchi as watchers

added Build master review-pending-level-1 labels

This will need a few runs of the CI. And this time I will also test changing of the project version...

This merge request affects 4 packages:

• Projects/AnalysisBase
• Projects/AthGeneration
• Projects/AthSimulation
• Projects/Athena

Adding @jchapman ,@akraszna ,@krumnack ,@vpascuzz ,@rbianchi as watchers

changed title from Fix for resusing of externals build to Fix for reusing of externals build

changed the description

CI Result SUCCESS (hash 0f230579)

	Athena	AthSimulation	AnalysisBase	AthGeneration
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
AnalysisBase: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 14659]

added 1 commit

• ba72bcad - TEST: Bump project version to 22.0.99

Compare with previous version

This merge request affects 6 packages:

• Projects/AnalysisBase
• Projects/AthDataQuality
• Projects/AthGeneration
• Projects/AthSimulation
• Projects/Athena
• Projects/VP1Light

Adding @jchapman ,@akraszna ,@krumnack ,@vpascuzz ,@rbianchi as watchers

added 15 commits

• ba72bcad...739e797f - 12 commits from branch atlas:master
• e4d22967 - Reuse existing externals build if possible
• d237ecf8 - Fix for reusing existing externals build
• ffb9072b - Do not use BINARY_TAG for externals stamp file

Compare with previous version

This merge request affects 4 packages:

• Projects/AnalysisBase
• Projects/AthGeneration
• Projects/AthSimulation
• Projects/Athena

Adding @jchapman ,@akraszna ,@krumnack ,@vpascuzz ,@rbianchi as watchers

This merge request affects 4 packages:

• Projects/AnalysisBase
• Projects/AthGeneration
• Projects/AthSimulation
• Projects/Athena

Adding @jchapman ,@akraszna ,@krumnack ,@vpascuzz ,@rbianchi as watchers

This merge request affects 4 packages:

• Projects/AnalysisBase
• Projects/AthGeneration
• Projects/AthSimulation
• Projects/Athena

Adding @jchapman ,@akraszna ,@krumnack ,@vpascuzz ,@rbianchi as watchers

resolved all threads

CI Result SUCCESS (hash ffb9072b)

	Athena	AthSimulation	AnalysisBase	AthGeneration
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
AnalysisBase: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 14715]

Looks fine, CI is good and all threads are resolved. Approving. Clara (L1)

added review-approved label and removed review-pending-level-1 label

merged

mentioned in commit 54a8d3f2