-
Notifications
You must be signed in to change notification settings - Fork 183
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
[ROOTMaster] Updated root to tip of branch master #9512
[ROOTMaster] Updated root to tip of branch master #9512
Conversation
A new Pull Request was created by @iarspider for branch IB/CMSSW_14_2_X/rootmaster. @aandvalenzuela, @cmsbuild, @iarspider, @smuzaffar can you please review it and eventually sign? Thanks. |
cms-bot internal usage |
please test with cms-sw/cmssw#46635 for CMSSW_14_2_ROOT6_X |
please abort |
Pull request #9512 was updated. |
please test with cms-sw/cmssw#46635 for CMSSW_14_2_ROOT6_X |
Pull request #9512 was updated. |
@iarspider , you also need to update dd4hep |
Did that already, thanks! |
-1 Failed Tests: Build The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: BuildI found compilation error when building: >> Checking EDM Class Version for src/CalibFormats/CaloObjects/src/classes_def.xml in libCalibFormatsCaloObjects.so >> Skipped running EDM Class Version test @@@@ ----> OK EDM Class Version >> Checking EDM Class Transients in libCalibFormatsCaloObjects.so @@@@ ----> OK EDM Class Transients gmake: *** [tmp/el8_amd64_gcc12/edm_checks/libAnalysisDataFormatsTrackInfo.so] Error 1 @@@@ ----> OK EDM Class Transients >> Checking EDM Class Version for src/CalibMuon/DTCalibration/test/stubs/classes_def.xml in libtestDTCalibration.so @@@@ ----> OK EDM Class Transients >> Skipped running EDM Class Version test >> Checking EDM Class Version for src/CalibPPS/AlignmentRelative/src/classes_def.xml in libCalibPPSAlignmentRelative.so |
test parameters:
|
please test |
-1 Failed Tests: UnitTests The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Unit TestsI found 4 errors in the following unit tests: ---> test TestFWCoreReflectionClassVersionUpdate had ERRORS ---> test TestIOPoolInputNoParentDictionary had ERRORS ---> test testFWCoreConcurrency had ERRORS and more ... Comparison SummarySummary:
|
src/FWCore/Reflection/test/TestFWCoreReflectionClassVersionUpdate and src/IOPool/Input/test/TestIOPoolInputNoParentDictionary - usual failures when ROOT version changes src/FWCore/Concurrency/test/testFWCoreConcurrency - timed out src/DQMServices/Demo/test/TestDQMGUIUpload - known failure cms-sw/cmssw#46682 @smuzaffar I think we can merge it, and keep an eye on testFWCoreConcurrency. |
looks good |
6f9b043
into
cms-sw:IB/CMSSW_14_2_X/rootmaster
Changes. Superseeds #9502