Analysis Externals Update, master branch (2020.02.25.)
This is a collection of a number of things, primarily aimed at making AnalysisBase
build/behave correctly on macOS.
The full list of changes in atlasexternals-2.0.58 wrt. atlasexternals-2.0.57 is (atlasexternals@2.0.57...2.0.58):
- Made the environment setup scripts work with path names containing spaces;
- Taught
AtlasCMake
how to handle the Apple Clang compiler/linker correctly; - Suppressed some long-standing warnings with CMake 3.9+ on macOS.
At the same time I made some cleanup in this repository as well:
- The projects had version number all over the place. I updated them all to the same (
22.0.10
) version. - Removed the usage of
<experimental/array>
from MuonStationIndex.cxx. Apple's compiler doesn't have that header anymore, and I couldn't see a reason why it should be included anyway.
Finally, to make the installed AnalysisBase release set up a correct runtime environment on macOS (or any other OS that provides Python 3 from the system), I had to set ATLAS_FORCE_PYTHON2=TRUE
for the project. For now... We'll have to see later on how to properly enable the building of both Python 2 and 3 versions of AnalysisBase
at the same time...
Pinging @krumnack.
Merge request reports
Activity
added 1 commit
- 2b10f27e - Made sure that analysis projects would find Python correctly.
added Build MuonSpectrometer master review-pending-level-1 labels
✅ CI Result SUCCESS (hash ee2a2317)Athena AthSimulation 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
📝 For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 9980]✅ CI Result SUCCESS (hash 2b10f27e)Athena AthSimulation 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
📝 For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 9984]@akraszna Do you want this to go in? Or leave it here for more comment? sorry we are short of reviewers this week.
Hi Adam,
I believe this is a fairly safe update. Not counting the fact that making both
Athena
andAthSimulation
have the same22.0.10
version may in fact introduce some issues in the current nightly. (For instance last night's nightlies were both installed under/cvmfs/atlas-nightlies.cern.ch/repo/sw/master/2020-02-26T2201
. With the version numbers synced up, we'll only have one GAUDI installation in such a case. This is being discussed in ATLINFR-3335...)But even with that, this update would help our developments on the AnalysisBase project. So it would be good to have this in.
😉 Cheers, Attila
mentioned in commit 763441f4
added sweep:ignore label