Skip to content
This repository was archived by the owner on Mar 18, 2019. It is now read-only.

Create universal wheels #10

Closed
wants to merge 1 commit into from
Closed

Create universal wheels #10

wants to merge 1 commit into from

Conversation

edmorley
Copy link
Contributor

@edmorley edmorley commented Aug 7, 2017

Previously the generated wheel was only for the version of Python under which the setup.py publish command was run, which means the Python 3 wheel was missing for the latest release:
https://pypi.python.org/pypi/coreschema/0.0.4

Previously the generated wheel was only for the version of Python
under which the `setup.py publish` command was run, which means the
Python 3 wheel was missing for the latest release:
https://pypi.python.org/pypi/coreschema/0.0.4
@jdufresne
Copy link

jdufresne commented Nov 5, 2017

After #14 lands, you could include the license file in the generated wheel using the license_file in setup.cfg:

[metadata]
license_file = LICENSE

See the documentation for information on this feature of wheel:

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

This would satisfy the license's requirement:

Redistributions of source code must retain the above copyright notice, this list of conditions and the following disclaimer.

@edmorley
Copy link
Contributor Author

edmorley commented Nov 5, 2017

Hi! That seems orthogonal to this PR? Wheels are already being created; this PR just makes sure they are created for both Python 2 and 3, not just one of Python {2, 3}.

As such, it's probably best as a new issue (or a comment on #14), rather than here.

@edmorley
Copy link
Contributor Author

@tomchristie, don't suppose you could merge this? :-)

@edmorley
Copy link
Contributor Author

Ping? :-)

@edmorley edmorley closed this Oct 3, 2018
@edmorley edmorley deleted the universal-wheel branch October 3, 2018 14:08
@tomchristie
Copy link
Contributor

tomchristie commented Oct 3, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants