Skip to content

[VISUALIZATION] [GCC16] cleanup for unused variables#50989

Closed
smuzaffar wants to merge 1 commit into
cms-sw:masterfrom
smuzaffar:visualization-gcc16-unsed-var
Closed

[VISUALIZATION] [GCC16] cleanup for unused variables#50989
smuzaffar wants to merge 1 commit into
cms-sw:masterfrom
smuzaffar:visualization-gcc16-unsed-var

Conversation

@smuzaffar
Copy link
Copy Markdown
Contributor

Cleanup set but unused variables which are causing build errors for GCC16 IBs https://cmssdt.cern.ch/SDT/cgi-bin/showBuildLogs.py/el9_amd64_gcc16/www/tue/17.0-tue-23/CMSSW_17_0_X_2026-05-19-2300

@cmsbuild
Copy link
Copy Markdown
Contributor

cmsbuild commented May 20, 2026

cms-bot internal usage

@cmsbuild
Copy link
Copy Markdown
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-50989/49404

@cmsbuild
Copy link
Copy Markdown
Contributor

A new Pull Request was created by @smuzaffar for master.

It involves the following packages:

  • Fireworks/Core (visualization)
  • Fireworks/ParticleFlow (visualization)

@Dr15Jones, @alja, @cmsbuild, @makortel can you please review it and eventually sign? Thanks.
@alja this is something you requested to watch as well.
@ftenchini, @mandrenguyen, @sextonkennedy you are the release manager for this.

cms-bot commands are listed here

@alja
Copy link
Copy Markdown
Contributor

alja commented May 20, 2026

+1

@cmsbuild
Copy link
Copy Markdown
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants