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

Remove now-obsolete NativeIO shim. #113

Closed
wants to merge 1 commit into from

Conversation

inklesspen
Copy link
Contributor

This was only needed for Python 2 compatibility, which was dropped in v1.16.0.

This was only needed for Python 2 compatibility, which was dropped in v1.16.0.
@arigo
Copy link
Contributor

arigo commented Aug 20, 2024

I'm not too keen on this kind of breaking change with no real purpose. It would be cool if I, for example, could still download the latest version and run it in python 2.7 with minimal fixes, and I know other people like me.

@arigo arigo closed this Aug 20, 2024
@inklesspen
Copy link
Contributor Author

Can you run it on 2.7, though? setup.py says the minimum python version is 3.8, and has done since v1.16.0

@inklesspen
Copy link
Contributor Author

Huh, I guess it does work on 2.7. Pip won't install it, but if you download the tarball and run setup.py it will happily install (although cffi/_shimmed_dist_utils.py uses some syntax that Python 2.7 doesn't support, when warning about Python 3.12 issues.)

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.

2 participants