PyUtils: Delay regex matching to capture static variables

As noticed by @tadej, when we enabled regex support in diff-root we lost the ability to exclude static variables under certain circumstances (see !37189 (closed)). We now delay the regex matching as late as possible to remedy this.

added bugfix urgent labels

This merge request affects 1 package:

• Tools/PyUtils

added Tools master review-pending-level-1 labels

Hi @tadej,

Could you give this a shot and see if it fits your use case? BTW, you can simplify your exclusion list to (of course this can be made more strict):

--ignore-leaves '.*detectorMask(0|1|2|3)'

to ignore:

xAOD::EventAuxInfo_v2_EventInfoAux.detectorMask0
xAOD::EventAuxInfo_v2_EventInfoAux.detectorMask1
xAOD::EventAuxInfo_v2_EventInfoAux.detectorMask2
xAOD::EventAuxInfo_v2_EventInfoAux.detectorMask3

Best, Serhan

CI Result FAILURE (hash 4e44184e)

	Athena	AthSimulation	AthGeneration	AnalysisBase
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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 21770]

I guess it does not work :(

added review-user-action-required label and removed review-pending-level-1 label

added 2 commits

• 69bd699a - Update default exclusion list
• fe235292 - Drop verbose output from diff-root in RunTier0Tests.py

Compare with previous version

This merge request affects 2 packages:

• Tools/PROCTools
• Tools/PyUtils

Adding @amete as watcher

added review-pending-level-1 label and removed review-user-action-required label

resolved all threads

added 1 commit

• d815e6b2 - Match the style in exclusion list parsing

Compare with previous version

This merge request affects 2 packages:

• Tools/PROCTools
• Tools/PyUtils

Adding @amete as watcher

CI Result SUCCESS (hash fe235292)

	Athena	AthSimulation	AthGeneration	AnalysisBase
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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 21782]

CI Result SUCCESS (hash d815e6b2)

	Athena	AthSimulation	AthGeneration	AnalysisBase
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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 21784]

Code changes look fine, approving. Pienpen (L1)

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

merged

mentioned in commit 9153bdc0

added sweep:ignore label