Skip to content

wheel builds: react to changes in pip's handling of build constraints#33

Merged
rapids-bot[bot] merged 2 commits intorapidsai:mainfrom
mmccarty:fix/new-build-constraint
Jan 20, 2026
Merged

wheel builds: react to changes in pip's handling of build constraints#33
rapids-bot[bot] merged 2 commits intorapidsai:mainfrom
mmccarty:fix/new-build-constraint

Conversation

@mmccarty
Copy link
Copy Markdown
Contributor

Description

Contributes to rapidsai/build-planning#242

Modifying ci/build_wheel.sh to

  • pass--build-constraint="${PIP_CONSTRAINT}" unless build isolation is enabled.
  • unset PIP_CONSTRAINT (set by rapids-init-pip)... it doesn't affect builds as of pip 25.3, and results in an error from pip wheel when set and --build-constraint is also passed

@mmccarty mmccarty requested a review from jameslamb January 16, 2026 21:46
@mmccarty mmccarty requested a review from a team as a code owner January 16, 2026 21:46
@mmccarty mmccarty added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Jan 16, 2026
@github-actions github-actions Bot added the ci label Jan 16, 2026
Comment thread ci/build_wheel.sh Outdated
@jameslamb
Copy link
Copy Markdown
Member

/ok to test

Copy link
Copy Markdown
Member

@jameslamb jameslamb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Double-checked logs and we seem to be getting what we want... no pip deprecation warnings and 100% cache hit rate building libcuforest.

Thanks!

@jameslamb
Copy link
Copy Markdown
Member

/merge

@rapids-bot rapids-bot Bot merged commit 04c66df into rapidsai:main Jan 20, 2026
78 checks passed
@jameslamb jameslamb deleted the fix/new-build-constraint branch January 20, 2026 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants