Skip to content

Restore support for Python 2.7.0-2.7.8#123

Merged
acdha merged 2 commits into
LibraryOfCongress:masterfrom
nsoranzo:patch-1
Apr 4, 2019
Merged

Restore support for Python 2.7.0-2.7.8#123
acdha merged 2 commits into
LibraryOfCongress:masterfrom
nsoranzo:patch-1

Conversation

@nsoranzo

@nsoranzo nsoranzo commented Dec 7, 2018

Copy link
Copy Markdown
Contributor

hashlib.algorithms_guaranteed was introduced only in Python 2.7.9, see https://docs.python.org/2.7/library/hashlib.html#hashlib.algorithms_guaranteed

Ubuntu 14.04 LTS (which will be supported until April 2019) still provides python 2.7.5 and therefore needs this fix.

Tested on python 2.7.3, 2.7.5, 2.7.12 and 3.5.3.

xref. https://github.com/LibraryOfCongress/bagit-python/pull/110/files/1e8c9102ca004ffe19984ac2eceaee8f0d616dfd#r238885995

`hashlib.algorithms_guaranteed` was introduced only in Python 2.7.9, see https://docs.python.org/2.7/library/hashlib.html#hashlib.algorithms_guaranteed

Ubuntu 14.04 LTS (which will be supported until April 2019) still provides python 2.7.5 and therefore needs this fix.

Tested on python 2.7.3, 2.7.5, 2.7.12 and 3.5.3.
@coveralls

coveralls commented Dec 8, 2018

Copy link
Copy Markdown

Coverage Status

Coverage decreased (-1.3%) to 82.252% when pulling 6b25e39 on nsoranzo:patch-1 into c39b650 on LibraryOfCongress:master.

@acdha
acdha merged commit 4b76c14 into LibraryOfCongress:master Apr 4, 2019
@acdha

acdha commented Apr 4, 2019

Copy link
Copy Markdown
Member

I merged this since it's small but also added a flag for the cleanup in #102

@nsoranzo
nsoranzo deleted the patch-1 branch April 4, 2019 13:55
@nsoranzo

nsoranzo commented Apr 4, 2019

Copy link
Copy Markdown
Contributor Author

Thanks @acdha!

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.

3 participants