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
Fix check for requirements (#7534), add missing dependencies #7537
Conversation
A new Pull Request was created by @iarspider for branch IB/CMSSW_12_3_X/master. @cmsbuild, @smuzaffar, @iarspider can you please review it and eventually sign? Thanks. |
please test |
There are some dependencies (build ones) that are not caught by |
Pull request #7537 was updated. |
Pull request #7537 was updated. |
@iarspider , yes please remove the change for
note that I think we can improve |
I have a |
@smuzaffar |
see #7538 for rust. |
Sure. I have a few more fixes (for build-time dependencies), will push them soon. |
Pull request #7537 was updated. |
Pull request #7537 was updated. |
please test |
@@ -1,3 +1,3 @@ | |||
Requires: py3-pybind11 py3-numpy py3-pydantic py3-python-rapidjson py3-rich | |||
BuildRequires: py3-setuptools-scm py3-scikit-build py3-cmake py3-make | |||
BuildRequires: py3-setuptools-scm py3-scikit-build cmake py3-make |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we need py3-make ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Technically, yes. Actually - not sure.
Pull request #7537 was updated. |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-676bf4/21696/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:
|
please test for slc7_ppc64le_gcc11 |
please test for slc7_aarch64_gcc11 |
please test for cs8_amd64_gcc11 |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-676bf4/21717/summary.html |
-1 Failed Tests: RelVals 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: RelVals
|
@smuzaffar I think it's ok to merge. The error doesn't seem to be related to PR |
+externals |
This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_12_3_X/master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-676bf4/21719/summary.html Comparison SummarySummary:
|
@smuzaffar I can move c823a9b (proposed fix for #7534) to a separate PR to avoid rebuilding unchanged python packages if necessary.