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
pick up root commits needed for modules testing #5558
pick up root commits needed for modules testing #5558
Conversation
A new Pull Request was created by @davidlange6 (David Lange) for branch IB/CMSSW_11_1_X/rootmodule. @cmsbuild, @smuzaffar, @mrodozov, @tulamor can you please review it and eventually sign? Thanks. |
please test |
The tests are being triggered in jenkins. |
whoops - I think I have more commits to pick up |
After root-project/root#4985 this module should work.
Pull request #5558 was updated. |
please test |
The tests are being triggered in jenkins. |
-1 Tested at: 3c2e1bd CMSSW: CMSSW_11_1_CXXMODULE_X_2020-02-13-2300 I found follow errors while testing this PR Failed tests: Build
I found compilation error when building: In file included from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_1_CXXMODULE_X_2020-02-13-2300/src/DataFormats/ParticleFlowReco/interface/RecoPFClusterRefCandidate.h:6: In file included from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_1_CXXMODULE_X_2020-02-13-2300/src/DataFormats/ParticleFlowReco/interface/PFCluster.h:12: In file included from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_1_CXXMODULE_X_2020-02-13-2300/src/DataFormats/ParticleFlowReco/interface/PFRecHit.h:19: In file included from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_11_1_CXXMODULE_X_2020-02-13-2300/src/Geometry/CaloGeometry/interface/CaloCellGeometry.h:7: In file included from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/slc7_amd64_gcc820/external/clhep/2.4.0.0-nmpfii/include/CLHEP/Geometry/Point3D.h:19: /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/slc7_amd64_gcc820/external/clhep/2.4.0.0-nmpfii/include/CLHEP/Vector/ThreeVector.h:41:7: error: redefinition of 'Hep3Vector' class Hep3Vector { ^ /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/slc7_amd64_gcc820/external/clhep/2.4.0.0-nmpfii/include/CLHEP/Vector/LorentzVector.h:30:10: note: '/data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/slc7_amd64_gcc820/external/clhep/2.4.0.0-nmpfii/include/CLHEP/Vector/ThreeVector.h' included multiple times, additional include site here #include "CLHEP/Vector/ThreeVector.h" ^ |
Comparison not run due to Build errors (RelVals and Igprof tests were also skipped) |
I've tested a full build on the side which works (at least compiles). |
and include cxxmodules.mk changes from @vgvassilev