Skip to content

Use specifiers to indicate dependence on ordereddict for python2.6 only#23

Merged
jaraco merged 1 commit intojaraco:masterfrom
asottile:patch-1
Jan 25, 2019
Merged

Use specifiers to indicate dependence on ordereddict for python2.6 only#23
jaraco merged 1 commit intojaraco:masterfrom
asottile:patch-1

Conversation

@asottile
Copy link
Copy Markdown
Contributor

Without this, a wheel built in python2 will have incorrect dependencies: - A wheel built with python2.7 will produce a py2-none-any wheel which is a proper installation candidate for a python2.6 pip, but this wheel will not have the ordereddict dependency - A wheel built with python2.6 will produce a py2-none-any wheel which is a proper installation candidate for python2.7 pip, but will unnecessarily install ordereddict.

To read more about these markers: - https://www.python.org/dev/peps/pep-0426/#environment-markers

For the version specifiers to be honored: - Install from a wheel (requires pip>1.4 (Released in 2013 (ancient)) - Install from a sdist (requires pip>=6 (Released December 2014))

Without this, a wheel built in python2 will have incorrect dependencies: - A wheel built with python2.7 will produce a py2-none-any wheel which is a proper installation candidate for a python2.6 pip, but this wheel will not have the ordereddict dependency - A wheel built with python2.6 will produce a py2-none-any wheel which is a proper installation candidate for python2.7 pip, but will unnecessarily install ordereddict.

To read more about these markers: - https://www.python.org/dev/peps/pep-0426/#environment-markers

For the version specifiers to be honored: - Install from a wheel (requires pip>1.4 (Released in 2013 (ancient)) - Install from a sdist (requires pip>=6 (Released December 2014))
@jaraco jaraco merged commit 5faa9fc into jaraco:master Jan 25, 2019
@asottile asottile deleted the patch-1 branch January 25, 2019 15:12
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.

2 participants