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 support for Python 2.x #1510

Merged
merged 19 commits into from
Jul 21, 2020
Merged

Remove support for Python 2.x #1510

merged 19 commits into from
Jul 21, 2020

Conversation

mhammond
Copy link
Owner

@mhammond mhammond commented May 2, 2020

I'm going to remove Python 2.x support, which will make things alot simpler. There are still lots of places that do, eg, sys.version_info < (3,) and I might get to some of these or I might not - it doesn't really matter if they hang around a bit longer.

@vernondcole
Copy link
Collaborator

This will surely make things easier, but I was expecting one last bug fix release before dropping 2.7 altogether. The adodbapi code, especially the documentation, could use some cleanup before release. I had to keep some ugly things in place for backwards compatibility reasons, and I can get rid of those now. How much time to I have to make things pretty?

@mhammond
Copy link
Owner Author

mhammond commented May 3, 2020

I can still make a release before landing this - what significant bugs since 227 make this worthwhile?

Re making things pretty, I'm not sure if you mean "how long between killing py2 and the next release"? If so, I'm fairly flexible :) Feel free to push relevant changes to this branch if you want, then all such cleanups could land at the same time.

@vernondcole
Copy link
Collaborator

vernondcole commented May 13, 2020 via email

@mhammond
Copy link
Owner Author

@vernondcole I've updated this branch and split some of the commits - the ones which change adodbapi are now in their own commits to make it easier for you to review. It would be great if you can have a look. I'm hoping to merge this within the next week or so, but don't currently have plans to make a release - so there will still be some time after merging this to update other things, such as docs etc. Let me know what you think.

@mhammond mhammond force-pushed the kill-py2 branch 2 times, most recently from 5f2687a to eda5f4f Compare June 24, 2020 05:02
Done via 2to3 with fixers:
  -f basestring -f exec -f except -f dict -f import -f imports -f next
  -f nonzero -f print -f raise -f raw_input -f long -f standarderror -f types
  -f unicode -f urllib -f xrange
Done via 2to3 with fixers:
  -f basestring -f exec -f except -f dict -f import -f imports -f next
  -f nonzero -f raise -f raw_input -f long -f standarderror -f types
  -f unicode -f urllib -f xrange -f future
@kxrob
Copy link
Collaborator

kxrob commented Jul 17, 2020

the execfile 2to3 is still missing: #1539 - error with PYTHONSTARTUP set.

@mhammond mhammond merged commit 9fbc2ed into master Jul 21, 2020
@mhammond mhammond deleted the kill-py2 branch July 21, 2020 22:30
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.

None yet

3 participants