This repository has been archived by the owner on May 5, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 29
tests from sdist not working #41
Comments
Closed
Ayup I'll take a look (can't give a deadline tho, sorry). Will try to merge it with the next version.
…--
Alex
----------------------------------------
14 Feb 2023 22:06:50 Sebastian Müller ***@***.***>:
Hi there,
the tests included in sdist are not working, since not all files are included and one gets the following error:
*FileNotFoundError: [Errno 2] No such file or directory: 'tests/numpy_shapes.npz'
...
FileNotFoundError: [Errno 2] No such file or directory: 'tests/samples.json.gz'
*
So I think, these two files need to be included explicitly in the sdist.
Cheers,
Sebastian
—
Reply to this email directly, view it on GitHub[#41], or unsubscribe[https://github.com/notifications/unsubscribe-auth/ABBDCXM23HEHPYNWPLYXCULWXPXWLANCNFSM6AAAAAAU4BR6Z4].
You are receiving this because you are subscribed to this thread.[Tracking image][https://github.com/notifications/beacon/ABBDCXKUKFPSADIRUV7VR7TWXPXWLA5CNFSM6AAAAAAU4BR6Z6WGG33NNVSW45C7OR4XAZNFJFZXG5LFVJRW63LNMVXHIX3JMTHF45VBCU.gif]
|
lmas
added a commit
that referenced
this issue
Feb 18, 2023
When building sdist, ONLY THIS FILE will pull in extra data. `package_data` in setup.py only works for bdist. Source: https://stackoverflow.com/a/14159430
@MuellerSeb I was unable to reproduce your issue, not sure what or how you tried to install the pkg. |
Merging change and closing due to inactivity. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Hi there,
the tests included in sdist are not working, since not all files are included and one gets the following error:
So I think, these two files need to be included explicitly in the sdist.
Cheers,
Sebastian
The text was updated successfully, but these errors were encountered: