[ROOT6] Updated root to tip of branch master #8312
Conversation
|
please test |
|
A new Pull Request was created by @aandvalenzuela (Andrea Valenzuela) for branch IB/CMSSW_13_1_X/rootmaster. @smuzaffar, @aandvalenzuela, @iarspider can you please review it and eventually sign? Thanks. |
|
-1 Failed Tests: Build BuildI found compilation error when building: #63 0x00002afae51840fd in WrapperCall (method=47257073078325, nargs=140726133274800, args_=0x7ffd5b2f3860, self=0x7ffd5b2f3850, result=0x7ffd5b2f3848) at /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/BUILD/el8_amd64_gcc11/lcg/root/6.29.01-0801b3cc330d6ebd235654131310dec3/root-6.29.01/bindings/pyroot/cppyy/cppyy-backend/clingwrapper/src/clingwrapper.cxx:774 =========================================================== Suggestion: You can run 'scram build updateclassversion' to generate src/DataFormats/BTauReco/src/classes_def.xml.generated with updated ClassVersion gmake: *** [tmp/el8_amd64_gcc11/edm_checks/libDataFormatsBTauReco.so] Error 1 >> Checking EDM Class Version for src/DataFormats/ForwardDetId/src/classes_def.xml in libDataFormatsForwardDetId.so @@@@ ----> OK EDM Class Version >> Checking EDM Class Transients in libDataFormatsFTLDigi.so @@@@ ----> OK EDM Class Version @@@@ ----> OK EDM Class Version |
|
test parameters:
|
|
please test |
cmsbuild
added
tests-started
tests-rejected
and removed
tests-rejected
tests-started
labels
|
-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 Transients in libDataFormatsCSCRecHit.so
@@@@ ----> OK EDM Class Version
>> Checking EDM Class Transients in libDataFormatsCTPPSDigi.so
@@@@ ----> OK EDM Class Version
>> Checking EDM Class Transients in libDataFormatsBeamSpot.so
/data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/el8_amd64_gcc11/lcg/root/6.29.01-0801b3cc330d6ebd235654131310dec3/etc/cling/std.modulemap:497:10: error: module 'std.bits/uses_allocator_args.h' requires feature 'cplusplus20'
module "bits/uses_allocator_args.h" {
^
/data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/el8_amd64_gcc11/external/gcc/11.2.1-f9b9dfdd886f71cd63f5538223d8f161/include/c++/11.2.1/memory_resource:42:10: note: submodule of top-level module 'std' implicitly imported here
#include // uninitialized_construct_using_alloc
^
|
|
Waiting for ROOT fix: root-project/root#12240 |
|
please test |
|
Pull request #8312 was updated. |
|
-1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-2c2cf9/30645/summary.html External BuildI found compilation warning when building: See details on the summary page. |
|
please test |
|
Pull request #8312 was updated. |
|
-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 Transients in libDataFormatsCTPPSDigi.so
@@@@ ----> OK EDM Class Version
>> Checking EDM Class Transients in libDataFormatsBeamSpot.so
@@@@ ----> OK EDM Class Version
>> Checking EDM Class Transients in libDataFormatsCTPPSReco.so
/data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/el8_amd64_gcc11/lcg/root/6.29.01-d211d0de98c4c1d92aca13cfeb930afd/etc/cling/std.modulemap:497:10: error: module 'std.bits/uses_allocator_args.h' requires feature 'cplusplus20'
module "bits/uses_allocator_args.h" {
^
/data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/el8_amd64_gcc11/external/gcc/11.2.1-f9b9dfdd886f71cd63f5538223d8f161/include/c++/11.2.1/memory_resource:42:10: note: submodule of top-level module 'std' implicitly imported here
#include // uninitialized_construct_using_alloc
^
|
|
Same error. Waiting for root-project/root#12240 |
|
@aandvalenzuela , looks like the patch here is not complete, can you please update it https://patch-diff.githubusercontent.com/raw/root-project/root/pull/12276.patch and restart the tests? |
|
Pull request #8312 was updated. |
|
please test |
|
Now it built! |
|
The new part of the patch is directly related to the errors.... Makes sense that this fixes things.
… On Feb 16, 2023, at 3:35 PM, Andrea Valenzuela ***@***.***> wrote:
Now it built!
FYI, @smuzaffar, @iarspider
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
|
|
-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 errors in the following unit tests: ---> test test_occi_std_length had ERRORS Comparison SummarySummary:
|
|
lets wait for root-project/root#12276 to go in and then update root master to use the latest commit. By the way, in future if you have to test a root PR then instead of including |
|
Update of root master with the tested patch at #8320 |
No news on #8289.
But let's run the tests just in case it is fixed.