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
Update yaml-cpp to 0.7.0 #7788
Update yaml-cpp to 0.7.0 #7788
Conversation
A new Pull Request was created by @smuzaffar (Malik Shahzad Muzaffar) for branch IB/CMSSW_12_4_X/master. @cmsbuild, @smuzaffar, @aandvalenzuela, @iarspider can you please review it and eventually sign? Thanks. |
please test for cs8_amd64_gcc10 |
please test for cs8_amd64_gcc10 |
please test for cs9_amd64_gcc11 |
-1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-789a79/23902/summary.html External BuildI found compilation error when building: make[2]: *** [test/prefix/googletest/CMakeFiles/gtest.dir/build.make:82: test/prefix/googletest/CMakeFiles/gtest.dir/src/gtest-all.cc.o] Error 1 make[2]: Leaving directory '/pool/condor/dir_256765/jenkins/workspace/ib-run-pr-tests/testBuildDir/BUILD/cs9_amd64_gcc11/external/yaml-cpp/0.7.0-49fbde0193b27de6becd287a01e9eb74/yaml-cpp-yaml-cpp-0.7.0' make[1]: *** [CMakeFiles/Makefile2:1124: test/prefix/googletest/CMakeFiles/gtest.dir/all] Error 2 make[1]: Leaving directory '/pool/condor/dir_256765/jenkins/workspace/ib-run-pr-tests/testBuildDir/BUILD/cs9_amd64_gcc11/external/yaml-cpp/0.7.0-49fbde0193b27de6becd287a01e9eb74/yaml-cpp-yaml-cpp-0.7.0' make: *** [Makefile:160: all] Error 2 error: Bad exit status from /pool/condor/dir_256765/jenkins/workspace/ib-run-pr-tests/testBuildDir/tmp/rpm-tmp.Okfjbl (%build) RPM build errors: line 37: It's not recommended to have unversioned Obsoletes: Obsoletes: external+yaml-cpp+0.7.0-49fbde0193b27de6becd287a01e9eb74 Bad exit status from /pool/condor/dir_256765/jenkins/workspace/ib-run-pr-tests/testBuildDir/tmp/rpm-tmp.Okfjbl (%build) |
please test for cs9_amd64_gcc11 |
Pull request #7788 was updated. |
please test |
hold |
Pull request has been put on hold by @smuzaffar |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-789a79/23909/summary.html 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: Comparison SummarySummary:
|
yaml-cpp is not required. So closing this PR |
Reverts #7787
need to test it for gcc11