-
Notifications
You must be signed in to change notification settings - Fork 115
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
python-rsa 4.1 dropped Python 2 support but did not add a python_requires>=3
stanza in setup.py
#152
Comments
Thanks for the report. Your proposed solution seems sensible, I think I can do that tomorrow. |
Hotfix related to sybrenstuvel/python-rsa#152.
Hotfix related to sybrenstuvel/python-rsa#152.
I just released 4.3 and 4.4.1. 4.3 is a re-tagged version 4.0, but I also took the opportunity to back-port two security fixes to Python 2.7. It does NOT support Python 3.4, as that caused some issues on Travis-CI. 4.4.1 i s a re-tagged release of version 4.2, with explicit support for Python 3.5 - 3.8. Both releases have an explicit |
Thanks a lot! Did this become 4.4 ? |
@sybrenstuvel Thanks for the quick update. From my local experiments, Copied from grpc/grpc#23197 |
I don't recommend yanking since it breaks build determinism for anyone using a build system like pipenv or poetry, but we need to fix this.. |
Echoing Lidi's comment, a Python 2 pip install lists up to 4.4
|
I released 4.4 too similar to 4.2, which caused the whole fix to crumble down, sorry about that. For now I have yanked 4.4, so Python 2.7 should pick 4.3, while not-dead Python versions should pick up on 4.4.1. I don't prefer yanking, as it indeed can cause some issues, but given that 4.4 has only been released a few hours, I hope that damage is minimal and preferred over releasing yet another two versions. Pretty please, with sugar on top, stop using this decade-old, end-of-life version of Python. |
Yanking didn't work, as the files are still listed on https://pypi.org/simple/rsa/. I re-tagged and re-released versions 4.5 (py27 compatible) and 4.6 (3.5+ compatible), and the mess has been cleaned up:
|
@sybrenstuvel any chance you can publish new wheels for 4.6 that aren't marked as universal? See #154 and the files listed here: https://pypi.org/project/rsa/4.6/#files |
I have deleted |
This change breaks users of
rsa
package who are still on Python 2 and who do not set an upper bound onrsa
to bersa<4.1
(or lower). It also breaks Python 2 users of any other library that depends on rsa with an open upper bound.A clean way to fix current situation would be to release a new version: (for example 4.3), using the sources of 4.0 (which are still Py2 compatible), and then release the most recent released sources in a follow-up release (4.4), and add a stanza
python_requires>=3
or perhapspython_requires>=3.5
(depending on which py3 versions you support).Note that yanking released versions may cause other breakages (see for example discussion in hamcrest/PyHamcrest#131), so a fix forward would be a cleaner way to address this.
The text was updated successfully, but these errors were encountered: