Skip to content

disallow fallback to Make in wheel builds#6286

Merged
rapids-bot[bot] merged 1 commit intorapidsai:branch-25.04from
jameslamb:no-make-fallback
Feb 3, 2025
Merged

disallow fallback to Make in wheel builds#6286
rapids-bot[bot] merged 1 commit intorapidsai:branch-25.04from
jameslamb:no-make-fallback

Conversation

@jameslamb
Copy link
Copy Markdown
Member

Contributes to rapidsai/build-planning#146

Proposes:

  • setting [tool.scikit-build].ninja.make-fallback = false, so scikit-build-core will not silently fallback to using GNU Make if ninja is not available
  • including all groups in dependencies.yaml with output_types: pyproject in output_types: requirements as well.

@jameslamb jameslamb added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jan 31, 2025
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented Jan 31, 2025

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@github-actions github-actions Bot added the Cython / Python Cython or Python issue label Jan 31, 2025
Comment thread dependencies.yaml
cuda_wheels:
specific:
- output_types: pyproject
- output_types: [requirements, pyproject]
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This should be safe: rapidsai/raft#2563 (comment)

@jameslamb
Copy link
Copy Markdown
Member Author

/ok to test

@jameslamb jameslamb changed the title WIP: disallow fallback to Make in wheel builds disallow fallback to Make in wheel builds Feb 3, 2025
@jameslamb jameslamb marked this pull request as ready for review February 3, 2025 14:00
@jameslamb jameslamb requested a review from a team as a code owner February 3, 2025 14:00
@jameslamb
Copy link
Copy Markdown
Member Author

/merge

@rapids-bot rapids-bot Bot merged commit b827040 into rapidsai:branch-25.04 Feb 3, 2025
@jameslamb jameslamb deleted the no-make-fallback branch February 3, 2025 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Cython / Python Cython or Python issue improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants