Skip to content
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

Drop support for python 2.7 #12

Open
aqeelat opened this issue Jul 16, 2023 · 2 comments
Open

Drop support for python 2.7 #12

aqeelat opened this issue Jul 16, 2023 · 2 comments

Comments

@aqeelat
Copy link

aqeelat commented Jul 16, 2023

I would like to work on this

@sebschrader
Copy link
Owner

As long as it's still possible with reasonable effort to build and test manylinux wheels for the EOL versions, I don't see the point of dropping support at this time. Although Python 2.7 is EOL, there are undoubtedly still many people using it (some distros with long support cycles are still shipping and supporting Python 2.7). I'd like to avoid actively breaking compatibility without much benefit. You may submit a PR, which I'm willing to review and merge into a preparation branch for future release though.

Because this is a C extension module and not a pure Python module the differences between Python 2/Python 3/Python 3 with PEP 489 (and also between CPython and PyPy) are resolved at compile-time by the pre-processor. I tried to minimize the scope and the quantity of the pre-processor #ifdef sections, to keep the maintenance burden low and the readability of the code high. The current unit tests wouldn't benefit much from dropped support either.

@aqeelat
Copy link
Author

aqeelat commented Jul 26, 2023

I agree with you, but people using python 2.7 will still be able to use older versions of the library. I believe python 2.7 should be abandoned and we should encourage/persuade users to move to python 3.7+.

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

No branches or pull requests

2 participants