Remove duplicate dependency from L1Trigger/CSCTriggerPrimitives BuildFile.xml

[fwyzard]

PR description:

Remove duplicate

<use name="Geometry/CSCGeometry"/>

entry from L1Trigger/CSCTriggerPrimitives/BuildFile.xml.

PR validation:

None.

[fwyzard]

please test

[cmsbuild]

The code-checks are being triggered in jenkins.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-31877/19265

• This PR adds an extra 12KB to repository

• There are other open Pull requests which might conflict with changes you have proposed:

  • File L1Trigger/CSCTriggerPrimitives/BuildFile.xml modified in PR(s): Remove option to run on single GEM pads; Simplify CSC trigger configuration #31631

[cmsbuild]

The tests are being triggered in jenkins.

• CMSSW_11_2_X_2020-10-20-1100/slc7_amd64_gcc820: https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/10154/console Started: 2020/10/20 22:32

[cmsbuild]

A new Pull Request was created by @fwyzard (Andrea Bocci) for master.

It involves the following packages:

L1Trigger/CSCTriggerPrimitives

@cmsbuild, @rekovic, @benkrikler can you please review it and eventually sign? Thanks.
@valuev, @Martin-Grunewald, @ptcox, @dildick this is something you requested to watch as well.
@silviodonato, @dpiparo, @qliphy you are the release manager for this.

cms-bot commands are listed here

[cmsbuild]

+1
Tested at: 6c7d122
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-7c24a8/10154/summary.html
CMSSW: CMSSW_11_2_X_2020-10-20-1100
SCRAM_ARCH: slc7_amd64_gcc820

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

• Migrate rest of CSCTFTrackProducer to EventSetup consumes #31801 @makortel: Migrate rest of CSCTFTrackProducer to EventSetup consumes
• Complete EventSetup migration for bunch of L1 modules #31763 @makortel: Complete EventSetup migration for bunch of L1 modules
• Removed unused fwkJobReports from MessageLogger in Alignment Subsystem #31852 @Dr15Jones: Removed unused fwkJobReports from MessageLogger in Alignment Subsystem
• Removed unused fwkJobReports from MessageLogger in Calibration Subsystem #31853 @Dr15Jones: Removed unused fwkJobReports from MessageLogger in Calibration Subsystem
• ES DQM: Enable concurrent processing of lumisections #31829 @tanmaymudholkar: ES DQM: Enable concurrent processing of lumisections
• Clean up a few dead assignments from CalibCalorimetry/HcalAlgos and RecoMuon/TrackingTools #31781 @perrotta: Clean up a few dead assignments from CalibCalorimetry/HcalAlgos and RecoMuon/TrackingTools
• remove hgcal local reco from reconstruction_trackingOnlyTask for trackingPhase2PU140 #31844 @slava77: remove hgcal local reco from reconstruction_trackingOnlyTask for trackingPhase2PU140
• Partially fix CLCT position bias after CCLUT (CCLUT-8) #31656 @dildick: Partially fix CLCT position bias after CCLUT (CCLUT-8)
• Removed unused fwkJobReports from MessageLogger from DQMServices Subsystem #31855 @Dr15Jones: Removed unused fwkJobReports from MessageLogger from DQMServices Subsystem
• fix stray Global Tags post GeometricDetExtra Migration #31871 @mmusich: fix stray Global Tags post GeometricDetExtra Migration
• Nano gen dqm #31591 @kdlong: Nano gen dqm
• Removed unused fwkJobReports from MessageLogger in GeneratorInterface Subsystem #31857 @Dr15Jones: Removed unused fwkJobReports from MessageLogger in GeneratorInterface Subsystem
• Run3-alca173 Use ESGetToken and LogVerbatim #31850 @bsunanda: Run3-alca173 Use ESGetToken and LogVerbatim
• Removed unused fwkJobReports from MessageLogger in Validation Subsystem #31862 @Dr15Jones: Removed unused fwkJobReports from MessageLogger in Validation Subsystem

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

[cmsbuild]

Comparison job queued.

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-7c24a8/10154/summary.html

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 4 differences found in the comparisons
• DQMHistoTests: Total files compared: 35
• DQMHistoTests: Total histograms compared: 2544100
• DQMHistoTests: Total failures: 7
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2544071
• DQMHistoTests: Total skipped: 22
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 4.653 KiB( 34 files compared)
• DQMHistoSizes: changed ( 1325.7 ): 4.653 KiB Physics/NanoAODDQM
• Checked 149 log files, 22 edm output root files, 35 DQM output files

[rekovic]

+1

[cmsbuild]

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

[silviodonato]

+1