Clean buildfiles after L1TriggerOffline in alphabet

[guitargeek]

PR description:

Another quick partially automatic BuildFile cleaning PR in the style of many before (for example #29485), this time covering all subsystems that come after Reco in the alphabet.

This one mainly focuses on the few subsystems that are between L1TriggerOffline and Reco in the alphabet. It also does what was promised in #29485 (comment): removing the library export of two header-only packages.

PR validation:

CMSSW compiles. It was checked that all newly added dependencies are actually required by the package with git grep.

[cmsbuild]

The code-checks are being triggered in jenkins.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-29541/14823

• This PR adds an extra 88KB to repository

[cmsbuild]

A new Pull Request was created by @guitargeek (Jonas Rembser) for master.

It involves the following packages:

CondCore/DBOutputService
MagneticField/Engine
MagneticField/GeomBuilder
MagneticField/Layers
MagneticField/ParametrizedEngine
MagneticField/Records
MagneticField/UniformEngine
MuonAnalysis/MomentumScaleCalibration
MuonAnalysis/MuonAssociators
OnlineDB/CSCCondDB
OnlineDB/EcalCondDB
OnlineDB/SiStripConfigDb
OnlineDB/SiStripESSources
OnlineDB/SiStripO2O
PerfTools/EdmEvent
PhysicsTools/CondLiteIO
PhysicsTools/FWLite
PhysicsTools/JetCharge
PhysicsTools/JetExamples
PhysicsTools/JetMCAlgos
PhysicsTools/MVAComputer
PhysicsTools/PatAlgos
PhysicsTools/PatExamples
PhysicsTools/RecoUtils
PhysicsTools/SelectorUtils
PhysicsTools/TensorFlow
PhysicsTools/UtilAlgos
PhysicsTools/Utilities
QCDAnalysis/Skimming
QCDAnalysis/UEAnalysis
RecoCaloTools/Selectors
RecoEgamma/EgammaElectronAlgos
RecoEgamma/EgammaElectronProducers
RecoTracker/TkNavigation
SimG4CMS/CherenkovAnalysis
SimG4CMS/HcalTestBeam
Validation/GlobalDigis

@perrotta, @smuzaffar, @andrius-k, @Dr15Jones, @makortel, @schneiml, @tlampen, @christopheralanwest, @civanch, @mdhildreth, @cmsbuild, @kmaeshima, @jfernan2, @fioriNTU, @slava77, @ggovi, @pohsun, @santocch, @tocheng can you please review it and eventually sign? Thanks.
@erikbutz, @rappoccio, @gouskos, @felicepantaleo, @jainshilpi, @hatakeyamak, @emilbols, @argiro, @ahinzmann, @lgray, @abbiendi, @varuns23, @seemasharmafnal, @mmarionncern, @JyothsnaKomaragiri, @makortel, @smoortga, @peruzzim, @jhgoh, @dgulhan, @jdolen, @HuguesBrun, @cericeci, @ferencek, @trocino, @Sam-Harper, @GiacomoSguazzoni, @rovere, @VinInn, @jdamgov, @bellan, @nhanvtran, @gkasieczka, @tocheng, @schoef, @mschrode, @ebrondol, @fabiocos, @clelange, @namapane, @riga, @battibass, @Fedespring, @calderona, @mverzett, @sobhatta, @lecriste, @afiqaize, @gpetruc, @mariadalfonso, @mmusich, @andrzejnovak, @folguera this is something you requested to watch as well.
@silviodonato, @dpiparo you are the release manager for this.

cms-bot commands are listed here

[perrotta]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/5832/console Started: 2020/04/23 08:07

[cmsbuild]

+1
Tested at: 4c3ebc2
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-dd15e8/5832/summary.html
CMSSW: CMSSW_11_1_X_2020-04-22-2300
SCRAM_ARCH: slc7_amd64_gcc820

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 8 differences found in the comparisons
• DQMHistoTests: Total files compared: 34
• DQMHistoTests: Total histograms compared: 2696435
• DQMHistoTests: Total failures: 3
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2696113
• DQMHistoTests: Total skipped: 319
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 33 files compared)
• Checked 147 log files, 16 edm output root files, 34 DQM output files

[jfernan2]

+1

[santocch]

+1

[slava77]

+1

for #29541 4c3ebc2

• technical; also addresses the point raised in Clean BuildFiles after Reco in alphabet #29485 in RecoTracker/TkNavigation/BuildFile.xml
• jenkins tests pass

[makortel]

+1

[civanch]

+1

[pohsun]

+1

[guitargeek]

Hi @silviodonato, just the same reminder as in #29295 (comment): the db-pending tag is not really used anymore for about least half a year, so probably this PR can be merged now.

[silviodonato]

merge
FYI @ggovi