New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove include of MessageLogger.h from DataFormats headers #32824
Conversation
FYI @davidlange6 |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-32824/21016
|
A new Pull Request was created by @Dr15Jones (Chris Jones) for master. It involves the following packages: DataFormats/EgammaCandidates @perrotta, @Martin-Grunewald, @slava77, @cmsbuild, @fwyzard, @jpata can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
-1 Failed Tests: Build HeaderConsistency ClangBuild BuildI found compilation error when building: >> Compiling edm plugin /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_3_X_2021-02-04-1100/src/RecoLocalTracker/SiStripRecHitConverter/plugins/SealModules.cc >> Compiling edm plugin /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_3_X_2021-02-04-1100/src/RecoLocalTracker/SiStripRecHitConverter/plugins/SiStripRecHitConverter.cc >> Compiling edm plugin /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_3_X_2021-02-04-1100/src/RecoLocalTracker/SiStripRecHitConverter/plugins/SiStripRecHitMatcherESProducer.cc >> Compiling edm plugin /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_3_X_2021-02-04-1100/src/RecoLocalTracker/SiStripRecHitConverter/plugins/StripCPEESProducer.cc /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_3_X_2021-02-04-1100/src/RecoLocalTracker/SiStripRecHitConverter/plugins/SiStripRecHitConverter.cc: In member function 'virtual void SiStripRecHitConverter::produce(edm::Event&, const edm::EventSetup&)': /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_3_X_2021-02-04-1100/src/RecoLocalTracker/SiStripRecHitConverter/plugins/SiStripRecHitConverter.cc:48:3: error: 'LogDebug' was not declared in this scope; did you mean 'gDebug'? 48 | LogDebug("SiStripRecHitConverter") << "found\n" | ^~~~~~~~ | gDebug gmake: *** [tmp/slc7_amd64_gcc900/src/RecoLocalTracker/SiStripRecHitConverter/plugins/RecoLocalTrackerSiStripRecHitConverterPlugins/SiStripRecHitConverter.cc.o] Error 1 >> Building edm plugin tmp/slc7_amd64_gcc900/src/RecoLocalTracker/SiStripRecHitConverter/plugins/RecoLocalTrackerSiStripRecHitConverterPlugins/libRecoLocalTrackerSiStripRecHitConverterPlugins.so Clang BuildI found compilation error while trying to compile with clang. Command used:
>> Compiling /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_3_X_2021-02-04-1100/src/DataFormats/TauReco/src/PFTau3ProngSummary.cc >> Compiling /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_3_X_2021-02-04-1100/src/DataFormats/TauReco/src/PFTauDecayMode.cc >> Compiling /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_3_X_2021-02-04-1100/src/DataFormats/TauReco/src/PFTauTagInfo.cc >> Compiling /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_3_X_2021-02-04-1100/src/DataFormats/TauReco/src/RecoTauPiZero.cc >> Compiling /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_3_X_2021-02-04-1100/src/DataFormats/TauReco/src/PFTauTransverseImpactParameter.cc /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_3_X_2021-02-04-1100/src/DataFormats/TauReco/src/PFTauDecayMode.cc:95:14: error: no member named 'LogError' in namespace 'edm' edm::LogError("PFTauDecayMode") ~~~~~^ 1 error generated. gmake: *** [tmp/slc7_amd64_gcc900/src/DataFormats/TauReco/src/DataFormatsTauReco/PFTauDecayMode.cc.o] Error 1 >> Building LCG reflex dict from header file src/DataFormats/TauReco/src/classes_1.h |
37fdb67
to
6eaa778
Compare
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-32824/21026
|
please test |
+1 |
+1 |
+1 |
+Upgrade |
+alca |
ping @cms-sw/analysis-l2 |
+1 |
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) |
+1 |
PR description:
None of the headers needed the include.
PR validation:
The code compiles.