Skip to content
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

File README.rst not included from sdist distribution #1

Closed
bbiskup opened this issue Jan 27, 2011 · 3 comments
Closed

File README.rst not included from sdist distribution #1

bbiskup opened this issue Jan 27, 2011 · 3 comments

Comments

@bbiskup
Copy link

bbiskup commented Jan 27, 2011

The file README.rst (referenced in setup.py) is not included when the sdist distribution is built. The following patch will add a MANIFEST.in file so README.rst will be included:

diff --git a/MANIFEST.in b/MANIFEST.in
new file mode 100644
index 0000000..bb37a27
--- /dev/null
+++ b/MANIFEST.in
@@ -0,0 +1 @@
+include *.rst
@AndreaCensi
Copy link
Owner

Thanks, fixed in devel branch.

@bbiskup
Copy link
Author

bbiskup commented Jan 27, 2011

Thanks for fixing it so quickly.
I have found another issue: the subdirectory "library" under "contracts" does not get included in the distribution package. I think the line
packages=['contracts']
in setup.py should be replaced by
packages=find_packages(".")
or (to avoid inclusion of the "testing" directory)
packages = ["contracts", "contracts.library"]

@AndreaCensi
Copy link
Owner

Thanks, fixed this as well.

I'll push 0.9.3 out in a couple of days, once I finish writing some documentation.

chowder referenced this issue in python-manylinux-wheels/contracts Jul 26, 2023
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants