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

[3.10] bpo-45436: Fix tkinter tests with Tcl/Tk 8.6.11+ (GH-29077) #29080

Merged
merged 1 commit into from
Oct 20, 2021

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Oct 20, 2021

Since v8.6.11, a few configuration options seem to accept an empty value
where they did not previously; particularly the type of a Menu
widget, and the compound of any ttk widget with a label. Providing an
explicit expected error message to checkEnumParam bypasses the check
of an empty value, which no longer raises TclError.
(cherry picked from commit 4fe454c)

Co-authored-by: Zachary Ware zach@python.org

https://bugs.python.org/issue45436

Since v8.6.11, a few configuration options seem to accept an empty value
where they did not previously; particularly the `type` of a `Menu`
widget, and the `compound` of any ttk widget with a label.  Providing an
explicit expected error message to `checkEnumParam` bypasses the check
of an empty value, which no longer raises `TclError`.
(cherry picked from commit 4fe454c)

Co-authored-by: Zachary Ware <zach@python.org>
@miss-islington
Copy link
Contributor Author

@zware: Status check is done, and it's a success ✅ .

2 similar comments
@miss-islington
Copy link
Contributor Author

@zware: Status check is done, and it's a success ✅ .

@miss-islington
Copy link
Contributor Author

@zware: Status check is done, and it's a success ✅ .

@ambv ambv merged commit b8dbb3a into python:3.10 Oct 20, 2021
@miss-islington miss-islington deleted the backport-4fe454c-3.10 branch October 20, 2021 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip news tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants