-
Notifications
You must be signed in to change notification settings - Fork 1k
wheel builds: react to changes in pip's handling of build constraints #21048
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
wheel builds: react to changes in pip's handling of build constraints #21048
Conversation
jameslamb
left a comment
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.
I think this is looking good! Checked the logs for all the wheel builds and saw the things I'd expect.
Similar to rapidsai/ucxx#567 (review):
- suggested an update to a code comment
- suggest we change the PR title to something that'll be more easily understood in the changelog:
"wheel builds: react to changes in pip's handling of build constraints"
If you agree with those, I can commit that comment and then admin-merge this to save us a CI cycle.
5f93f3b to
fa5d1dd
Compare
Co-authored-by: James Lamb <jaylamb20@gmail.com>
fa5d1dd to
6ca68c1
Compare
jameslamb
left a comment
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.
re-approving, I think everything is looking great, thanks!
|
/merge |
c6e9a21
into
rapidsai:release/26.02
Description
Contributes to rapidsai/build-planning#242
Modifying
ci/build_wheel.shto avoid passing--build-constraintand--no-build-isolationtogether which results in an error frompip, however we want to keep environment variablePIP_CONSTRAINTset unconditionally.Can be merged after rapidsai/gha-tools#237