Externals Update, master branch (2022.10.04.)
Updated all projects to atlasexternals-2.0.141. The "one" update wrt. atlasexternals-2.0.140 is (atlasexternals@2.0.140...2.0.141):
- Updated CheckerGccPlugins with the following (thanks to @ssnyder):
- Skip checking of
_rootcint.cxx
files. - Replace the hardcoded list of directory names for code that is to be considered thread-safe with a configuration parameter.
- For warnings about calling non-const methods via a pointer member from a const method, realize that
unique_ptr::get()
also returns the pointer. Treat it likeoperator->
. Also fix an issue with suppressing this warning forunique_ptr
. - Don't warn about overriding the non-const
execute()
method ofAlgorithm
with a non-thread-safe method. - Fix warning suppression for ctor calls.
- Remove references to
ATLAS_CTORDOR_NOT_THREAD_SAFE
workaround. - Warn about writing via a pointer member in a const member function.
- Skip checking of
This update technically does not require full-build. But in order to make sure that no build warnings are introduced with the updated code checker, let's use it anyway. (It means that merging this MR is safe at any point. As long as the CI succeeds...)
Pinging @fwinkl.
Merge request reports
Activity
added full-build label
This merge request affects 9 packages:
- Projects/AnalysisBase
- Projects/AthAnalysis
- Projects/AthDataQuality
- Projects/AthDerivation
- Projects/AthGeneration
- Projects/AthSimulation
- Projects/Athena
- Projects/DetCommon
- Projects/VP1Light
Affected files list will not be printed in this case
Adding @krumnack ,@jchapman ,@akraszna ,@rbianchi as watchers
added Build master review-pending-level-1 labels
CI Result SUCCESS (hash 9bee89a6)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
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
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 59794]removed review-pending-level-1 label
added review-approved label
mentioned in commit f0da3f58
added sweep:ignore label