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
Fix compatibility with setuptools 39. #9
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
@@ -16,14 +16,10 @@ | |||
from setuptools import package_index | |||
|
|||
|
|||
_final_parts = '*final-', '*final' | |||
def _final_version(parsed_version): | |||
"""Function copied from zc.buildout.easy_install._final_version |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the docstring still accurate now that you've replaced the implementation?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes it is, I copied the implementation from the current version of zc.buildout
.
trunked_candidate = [x for x in dist.parsed_version[:level] | ||
if not x.startswith('*')] | ||
trunked_current = parsed_version.base_version.split('.')[:level] | ||
trunked_candidate = dist.parsed_version.base_version.split('.')[:level] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The comment needs some updating -- the "remove *final" bit is no longer true.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch!
Let's drop Python 2.6 from travis (and tox and setup.py and mention that in the changelog). (We should also add 3.5 and 3.6 to travis and tox and setup.py and changelog.) |
@mgedmin I dropped 2.6 and 3.3. |
@mgedmin Could you create a release on PyPI or give me ( |
Fixes #8.