Added support for <foreach/for set="var" value="v1,v2"> + cleanup

[smuzaffar]

Cleanup for BuildFile and SimpleDoc parsing. Added support for

<foreach set="i" value="val1,val2">
  <for set="j" value="1,2,1"/>
    <test name="test_${i}_{j}" command="true"/>
  </for>
</foreach>

which should expand to

<test name="test_val1_1" command="true"/>
<test name="test_val1_2" command="true"/>
<test name="test_val2_1" command="true"/>
<test name="test_val2_2" command="true"/>

this can be used for adding tests based on the gpu backends e.g

<foreach set="gpu" value="cuda,rocm">
  <iftool name="pytorch-${gpu}">
    <test name="test-pytorch-${gpu}" command="pytorch-test.py ${gpu}">
     <use name="${gpu}"/>
    </test>
  </iftool>
</foreach>

[cmsbuild]

A new Pull Request was created by @smuzaffar for branch SCRAMV3.

@akritkbehera, @cmsbuild, @iarspider, @raoatifshad, @smuzaffar can you please review it and eventually sign? Thanks.
@ftenchini, @mandrenguyen, @sextonkennedy you are the release manager for this.
cms-bot commands are listed here

[cmsbuild]

cms-bot internal usage

[smuzaffar]

please test using full cmssw

[cmsbuild]

Pull request #89 was updated.

[cmsbuild]

Pull request #89 was updated.

[cmsbuild]

Pull request #89 was updated.

[smuzaffar]

please test using full cmssw

[smuzaffar]

please test using full cmssw for CMSSW_12_4_X

[smuzaffar]

please test using full cmssw for CMSSW_14_0_X

[smuzaffar]

please test using full cmssw for CMSSW_16_1_X

[cmsbuild]

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-7dae90/53873/summary.html
COMMIT: 256131a
CMSSW: CMSSW_12_4_X_2026-06-07-0000/el8_amd64_gcc10
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/SCRAM/89/53873/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

• You potentially removed 951 lines from the logs
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 51
• DQMHistoTests: Total histograms compared: 3766083
• DQMHistoTests: Total failures: 2
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3766059
• DQMHistoTests: Total skipped: 22
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 50 files compared)
• Checked 212 log files, 167 edm output root files, 51 DQM output files
• TriggerResults: no differences found

[cmsbuild]

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-7dae90/53870/summary.html
COMMIT: 256131a
CMSSW: CMSSW_20_1_X_2026-06-11-1100/el9_amd64_gcc13
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/SCRAM/89/53870/install.sh to create a dev area with all the needed externals and cmssw changes.

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:

• @bsunanda Phase2-hgx367 Try to create a scenario with T37, C27, N17, I22, F9 cmssw#51080
• @cms-ngt-hlt Minor adjustments for ngtScouting-modified ph2 HLT menus cmssw#51186

You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-7dae90/53870/git-recent-commits.json
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-7dae90/53870/git-merge-result

Comparison Summary

Summary:

• You potentially removed 166 lines from the logs
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 44
• DQMHistoTests: Total histograms compared: 3267025
• DQMHistoTests: Total failures: 58
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3266949
• DQMHistoTests: Total skipped: 18
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 43 files compared)
• Checked 191 log files, 159 edm output root files, 44 DQM output files

[cmsbuild]

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-7dae90/53874/summary.html
COMMIT: 256131a
CMSSW: CMSSW_14_0_X_2026-06-07-0000/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/SCRAM/89/53874/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

• You potentially removed 178 lines from the logs
• Reco comparison results: 512 differences found in the comparisons
• DQMHistoTests: Total files compared: 48
• DQMHistoTests: Total histograms compared: 3349879
• DQMHistoTests: Total failures: 355
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3349504
• DQMHistoTests: Total skipped: 20
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 47 files compared)
• Checked 202 log files, 165 edm output root files, 48 DQM output files
• TriggerResults: no differences found

[cmsbuild]

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-7dae90/53875/summary.html
COMMIT: 256131a
CMSSW: CMSSW_16_1_X_2026-06-11-1100/el8_amd64_gcc13
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/SCRAM/89/53875/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

• You potentially removed 181 lines from the logs
• Reco comparison results: 5 differences found in the comparisons
• DQMHistoTests: Total files compared: 53
• DQMHistoTests: Total histograms compared: 4182605
• DQMHistoTests: Total failures: 67
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 4182518
• DQMHistoTests: Total skipped: 20
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 52 files compared)
• Checked 227 log files, 197 edm output root files, 53 DQM output files
• TriggerResults: no differences found

[smuzaffar]

+externals

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next SCRAMV3 IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @sextonkennedy, @ftenchini, @mandrenguyen (and backports should be raised in the release meeting by the corresponding L2)