-
-
Notifications
You must be signed in to change notification settings - Fork 778
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 _depends.json
not included in wheel
#7578
Conversation
a90351d
to
07472e0
Compare
/test mini |
Q: Is this something that we should also include in the conda package? |
/test mini |
It's nice to have. CuPy ignores it silently if it is missing. This PR let |
Test failure is irrelevant. |
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.
LGTM!
Fix `_depends.json` not included in wheel
Confirmed: conda-forge/cupy-feedstock#218 (comment) |
Thanks for checking, Leo! |
The feature added in #7279 was not working in shipped wheels as the
_depends.json
file is not included in the wheel.c.f. pypa/setuptools#1064