Add CLUE external library.

[rovere]

This addresses the issue here.

[cmsbuild]

A new Pull Request was created by @rovere for branch IB/CMSSW_14_1_X/master.

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

[cmsbuild]

cms-bot internal usage

[fwyzard]

please test

[smuzaffar]

@rovere , can you please rebase your changes on top of latest IB/CMSSW_14_1_X/master branch and add clue dependency in cmssw-tools.spec.

[cmsbuild]

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-d80f8e/39802/summary.html
COMMIT: 2dea0cd
CMSSW: CMSSW_14_1_X_2024-06-09-0000/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmsdist/9231/39802/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

• You potentially added 3 lines to the logs
• Reco comparison results: 9 differences found in the comparisons
• DQMHistoTests: Total files compared: 48
• DQMHistoTests: Total histograms compared: 3338974
• DQMHistoTests: Total failures: 9
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3338945
• 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]

Pull request #9231 was updated.

[cmsbuild]

Pull request #9231 was updated.

[rovere]

@smuzaffar let me know if that's what you asked for.

[fwyzard]

I don't remember why alpaka is in the Linux-only section, but could you move clue to the same spot, right after alpaka itself ?

[rovere]

@fwyzard you are right, since clue depends on alpaka.
Did it, thanks.

[cmsbuild]

Pull request #9231 was updated.

[smuzaffar]

@rovere , can you please also change this to

%define git_commit V_%(echo %{realversion} | tr '.' '_')

this way in future we only need to update the version in the first line :-)

[rovere]

Done, thanks!

[fwyzard]

You could also change the release tag in GitHub to 1.0.0 :-)

[smuzaffar]

@smuzaffar let me know if that's what you asked for.

looks good

[cmsbuild]

Pull request #9231 was updated.

[fwyzard]

please test

[cmsbuild]

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-d80f8e/39813/summary.html
COMMIT: afa756e
CMSSW: CMSSW_14_1_X_2024-06-10-1100/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmsdist/9231/39813/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

• You potentially added 15 lines to the logs
• Reco comparison results: 9 differences found in the comparisons
• DQMHistoTests: Total files compared: 48
• DQMHistoTests: Total histograms compared: 3338974
• DQMHistoTests: Total failures: 851
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3338103
• 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]

REMINDER @rappoccio, @sextonkennedy, @antoniovilela: This PR was tested with cms-sw/cmssw#45178, please check if they should be merged together

[srimanob]

Hi @rappoccio @antoniovilela
Should this external PR also be merged? Thanks.

[fwyzard]

Yes, this external is needed to go with cms-sw/cmssw#45178 :

RecoLocalCalo/HGCalRecProducers/plugins/BuildFile.xml, line 49

  <use name="clue"/>

[smuzaffar]

+externals

@cms-sw/orp-l2 , I am merging this for 14.1.X as it is needed by which is already merged

[cmsbuild]

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