-
-
Notifications
You must be signed in to change notification settings - Fork 933
2.0.9 fails on Python 2.6 #540
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
Comments
PR #541 was merged on 8 Dec 2016. Can this issue be closed? Whilst on the topic, is Python 2.6 still something you need to support, or might it be time to consider dropping it more fully? Here's the pip installs for GitPython from PyPI for last month:
Source: |
@hugovk Thanks a lot for publishing some hard facts about its usage here! I wasn't even aware of download information once again being available. I think you are providing a strong argument to indeed drop python 2.6 support, the next time we break it. Alternatively, if someone feels strongly about getting rid of it to improve the code climate for example, PRs are always welcome. As the PR mentioned in this issue was merged, I think it's fair to close this ticket, too. |
Yeah, there's quite a lot of interesting stuff you can now get out of PyPI and pypinfo: Here's some examples: $ pypinfo GitPython country
Served from cache: False
Data processed: 8.22 GiB
Data billed: 8.22 GiB
Estimated cost: $0.05
| country | download_count |
| ------- | -------------- |
| US | 548,405 |
| IE | 13,000 |
| DE | 12,170 |
| FR | 8,712 |
| GB | 7,789 |
| FI | 7,775 |
| CA | 4,966 |
| EU | 3,961 |
| NL | 3,569 |
| CN | 3,337 |
$ pypinfo GitPython system distro
Served from cache: False
Data processed: 13.59 GiB
Data billed: 13.59 GiB
Estimated cost: $0.07
| system_name | distro_name | download_count |
| ----------- | ------------------------------- | -------------- |
| Linux | Ubuntu | 405,741 |
| Linux | CentOS Linux | 87,623 |
| Linux | Debian GNU/Linux | 48,369 |
| Linux | None | 25,180 |
| None | None | 23,718 |
| Linux | Red Hat Enterprise Linux Server | 7,750 |
| Linux | Amazon Linux AMI | 7,315 |
| Windows | None | 6,645 |
| Linux | Alpine Linux | 6,477 |
| Darwin | macOS | 5,857 | |
Super-interesting indeed! I wonder how these numbers are actually computed (just a rhetorical question really), just because they seem very, very high! Especially since it's just the monthly downloads. Could GitPython really be that popular :D? |
I was thankful to see your reaction to my issue #453, but unfortunately GitPython 2.0.9 has some issues on Python 2.6. GitPython 2.0.8 worked like a charm for us.
The issues are:
git/cmd.py
around line 800 (dict comprehensions were introduced in py27).unittest.case
ingit/util.py
and in several test cases (these were added tounittest
in py27). Also, the Pythonunittest
docs suggest they be imported fromunittest
and not fromunittest.case
, but that is minor.I have fixed those locally and will create a PR for your review.
The text was updated successfully, but these errors were encountered: