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

Add license file to source distribution #137

Closed

Conversation

synapticarbors
Copy link

Resolves #91

@lambacck
Copy link
Contributor

This includes the LICENSE.txt but not other files like CONTRIBUTORS.txt. It also doesn't exclude the .tox directory. Additionally LICENSE.txt isn't included in the whl file during python setup.py bdist_wheel. I have also been working on altering MANIFEST.in. This is what I currently have:

include *.rst
include *.txt
include *.cfg
include *.py
include *.ini
include regen-docs
recursive-include docs *.html *.css
recursive-include formencode/javascript *.js
recursive-include formencode/i18n *.py *.pot *.po *.mo
recursive-include formencode/tests *.txt
recursive-include examples *

prune docs/_build
prune .git
prune .tox
prune build
prune dist

Any chance you know how to include/exclude all the things?

@synapticarbors
Copy link
Author

I'm not sure about the wheel issue as I really haven't built any previously. The only thing I could find is the following stackoverflow answer, https://stackoverflow.com/a/39823590/392949, but it seems like you're already using include_package_data=True in setup() of the setup.py. So unfortunately I'm not sure. I tried to build the wheel, but examining its content, I didn't see the LICENSE.txt file either.

@Cito
Copy link
Member

Cito commented Jan 14, 2018

Maybe setting this in setup.cfg will help:

[metadata]
license_file = LICENSE.txt

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

Successfully merging this pull request may close these issues.

Include LICENSE file in the distribution.
3 participants