Skip to content

Use pyproject for auto discovery of packages and data #220

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

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

vzhurba01
Copy link
Collaborator

This PR adopts the same auto-discovery style as the cuda.core counterpart. This amendment is assured to expose the same data file as the posted Linux 12.6.2 patch. The change in style assist in getting most of the Windows files through.

Know issue: Windows builds can't auto-discover the trampoline files and they remain omitted from the final distributable. Note that Linux with this change does not face the same issue.

@vzhurba01 vzhurba01 added bug Something isn't working P0 High priority - Must do! cuda.bindings Everything related to the cuda.bindings module labels Nov 6, 2024
@vzhurba01 vzhurba01 added this to the cuda-python 12.6.2 & 11.8.5 milestone Nov 6, 2024
@vzhurba01 vzhurba01 self-assigned this Nov 6, 2024
Copy link
Contributor

copy-pr-bot bot commented Nov 6, 2024

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Copy link
Member

@leofang leofang left a comment

Choose a reason for hiding this comment

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

Thanks, Vlad! So this fixes the Windows build, and has no effect on the Linux build (so that the hot fixes pushed out are fine)?

@vzhurba01 vzhurba01 merged commit f953ab3 into NVIDIA:main Nov 6, 2024
@vzhurba01
Copy link
Collaborator Author

Correct. The hot fixes pushed out are fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cuda.bindings Everything related to the cuda.bindings module P0 High priority - Must do!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants