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 MessageLoggerHeader from SimDataFormats headers #32847
Remove MessageLoggerHeader from SimDataFormats headers #32847
Conversation
Exceptions are already being used in other constructors of MixCollection
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-32847/21061
|
A new Pull Request was created by @Dr15Jones (Chris Jones) for master. It involves the following packages: PhysicsTools/HepMCCandAlgos @cmsbuild, @civanch, @santocch, @mdhildreth can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
-1 Failed Tests: Build BuildI found compilation error when building: >> Compiling /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_3_X_2021-02-08-1100/src/SimMuon/GEMDigitizer/src/ME0PreRecoGaussianModel.cc >> Compiling /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_3_X_2021-02-08-1100/src/SimMuon/GEMDigitizer/src/ME0PreRecoNoSmearModel.cc >> Compiling /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_3_X_2021-02-08-1100/src/SimMuon/GEMDigitizer/src/ME0SimpleModel.cc >> Compiling /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_3_X_2021-02-08-1100/src/SimMuon/GEMDigitizer/src/SealModule.cc /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_3_X_2021-02-08-1100/src/SimMuon/GEMDigitizer/src/ME0SimpleModel.cc: In member function 'int ME0SimpleModel::getSimHitBx(const PSimHit*, CLHEP::HepRandomEngine*)': /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_3_X_2021-02-08-1100/src/SimMuon/GEMDigitizer/src/ME0SimpleModel.cc:135:5: error: 'LogDebug' was not declared in this scope; did you mean 'gDebug'? 135 | LogDebug("ME0SimpleModel") << "checktime " | ^~~~~~~~ | gDebug gmake: *** [tmp/slc7_amd64_gcc900/src/SimMuon/GEMDigitizer/src/SimMuonGEMDigitizer/ME0SimpleModel.cc.o] Error 1 >> Building shared library tmp/slc7_amd64_gcc900/src/SimMuon/GEMDigitizer/src/SimMuonGEMDigitizer/libSimMuonGEMDigitizer.so |
5465d63
to
35640b1
Compare
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-32847/21080
|
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-f8497e/12802/summary.html Comparison SummarySummary:
|
+1 |
@cms-sw/analysis-l2 @cms-sw/upgrade-l2 |
+Upgrade |
+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:
Removed the use of FWCore/MessageLogger/interface/MessageLogger.h from SimDataFormats headers. Only one of the headers actually used MessageLogger and for that one it was actually more consistent to throw an exception instead.
PR validation:
The code and all of its dependencies compile.