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

Include license file in the generated wheel packages #1207

Closed
wants to merge 1 commit into from
Closed

Include license file in the generated wheel packages #1207

wants to merge 1 commit into from

Conversation

jdufresne
Copy link
Contributor

The wheel package format supports including the license file. This is
done using the [metadata] section in the setup.cfg file. For additional
information on this feature, see:

https://wheel.readthedocs.io/en/stable/index.html#including-the-license-in-the-generated-wheel-file

Helps satisfy the project's license:

Redistributions in binary form must reproduce the above copyright
notice, this list of conditions and the following disclaimer in the
documentation and/or other materials provided with the distribution.

@giampaolo
Copy link
Owner

Isn't there a way to do this without adding another file? The root dir is already pretty crowded.

@jdufresne
Copy link
Contributor Author

I'm not aware of any other method.

Are you perhaps thinking of MANIFEST.in? That file does not affect wheels, only source distributions.

If you have some other suggestion, let me know, but I don't think one exists.

The wheel package format supports including the license file. This is
done using the [metadata] section in the setup.cfg file. For additional
information on this feature, see:

https://wheel.readthedocs.io/en/stable/index.html#including-the-license-in-the-generated-wheel-file

Helps satisfy the project's license:

> Redistributions in binary form must reproduce the above copyright
> notice, this list of conditions and the following disclaimer in the
> documentation and/or other materials provided with the distribution.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants