bpo-45873: Restore Python 3.6 compatibility (GH-29730) by tiran · Pull Request #29730 · python/cpython
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but is it even worth testing for 3.7+? Why not always take the slow path?
LGTM, but is it even worth testing for 3.7+? Why not always take the slow path?
I left the old implementation in because it's more readable. The fast path is now in a doc string for future reference.
I also added more debug code to configure to figure out what is going on on the Debian buildbot.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(Why do we even bother with "python3 python"? For weird platforms that have only one of those but it happens to be 3.6 or better?)
(Why do we even bother with "python3 python"? For weird platforms that have only one of those but it happens to be 3.6 or better?)
I don't know and kept python 3 python just in case.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:-)
@tiran: Status check is done, and it's a failure ❌ .
tiran
changed the title
bpo-45873: Restore Python 3.6 compatibility
bpo-45873: Restore Python 3.6 compatibility (GH-29730)
tiran
deleted the
bpo-45873-py36
branch
remykarem pushed a commit to remykarem/cpython that referenced this pull request
Co-authored-by: Guido van Rossum <gvanrossum@gmail.com>