Skip to content

Conversation

@aandvalenzuela
Copy link
Contributor

No description provided.

@aandvalenzuela aandvalenzuela requested a review from iarspider July 19, 2022 06:57
@aandvalenzuela
Copy link
Contributor Author

please test

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @aandvalenzuela (Andrea Valenzuela) for branch IB/CMSSW_12_5_X/rootmaster.

@smuzaffar, @aandvalenzuela, @iarspider can you please review it and eventually sign? Thanks.
@perrotta, @dpiparo, @qliphy, @rappoccio you are the release manager for this.
cms-bot commands are listed here

@cmsbuild
Copy link
Contributor

-1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-b27a64/26318/summary.html
COMMIT: 8ef0a51
CMSSW: CMSSW_12_5_ROOT6_X_2022-07-18-2300/el8_amd64_gcc10
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmsdist/7996/26318/install.sh to create a dev area with all the needed externals and cmssw changes.

External Build

I found compilation error when building:

make[2]: *** [Makefile:672: all-local] Error 1
make[2]: Leaving directory '/data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/BUILD/el8_amd64_gcc10/external/yoda/1.9.5-c9c6ee086869cf5cf7bdf464908a54f9/yoda-1.9.5/pyext'
make[1]: *** [Makefile:397: all-recursive] Error 1
make[1]: Leaving directory '/data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/BUILD/el8_amd64_gcc10/external/yoda/1.9.5-c9c6ee086869cf5cf7bdf464908a54f9/yoda-1.9.5/pyext'
make: *** [Makefile:519: all-recursive] Error 1
error: Bad exit status from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/tmp/rpm-tmp.m4jPEC (%build)


RPM build errors:
line 37: It's not recommended to have unversioned Obsoletes: Obsoletes: external+yoda+1.9.5-c9c6ee086869cf5cf7bdf464908a54f9
Macro expanded in comment on line 332: %{pkginstroot}/${PYTHON3_LIB_SITE_PACKAGES}


@aandvalenzuela
Copy link
Contributor Author

We believe the error is due to root-project/root#10759 and it will be fixed once root-project/root#10987 is merged.

@aandvalenzuela
Copy link
Contributor Author

Closing this PR and starting a fresh new one in #7999

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants