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
bpo-46996: Remove support of Tcl/Tk < 8.5.12 #31839
bpo-46996: Remove support of Tcl/Tk < 8.5.12 #31839
Conversation
If you want to schedule another build, you need to add the " |
Lib/test/test_tcl.py
Outdated
v = get_tk_patchlevel() | ||
if v >= (8, 6, 0, 'final') or (8, 5, 8) <= v < (8, 6): | ||
integers += (2**63, -2**63-1, 2**1000, -2**1000) | ||
integers += (2**63, -2**63-1, 2**1000, -2**1000) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If a condition is unlocked, maybe there is a sense to group numbers into (N-1, -N, N, -N-1).
def get_integers(self):
return (0, 1, -1,
2**31-1, -2**31, 2**31, -2**31-1,
2**63-1, -2**63 ,2**63, -2**63-1,
2**1000, -2**1000)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought about this, but kept simpler diff because there is already many changes in this PR.
But if you also suggest this I'll rewrite this code.
Thanks, Serhiy! Do you want to merge this before GH-31698? |
What is better for you? Does this PR need some changes in the build scripts or you will add them in #31698? |
Any order is fine with me. GH-31698 requires only minimal adjustments after this has been merged, so we can go for this first. |
https://bugs.python.org/issue46996