Skip to content

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

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

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 [email protected]

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 <[email protected]>
@miss-islington
Copy link
Contributor Author

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

4 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 ✅ .

@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 2ce3816 into python:3.8 Oct 20, 2021
@miss-islington miss-islington deleted the backport-4fe454c-3.8 branch October 20, 2021 14:08
bcail pushed a commit to bcail/cpython that referenced this pull request Apr 6, 2024
…thonGH-29093)

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 <[email protected]>
bcail pushed a commit to bcail/cpython that referenced this pull request Apr 9, 2024
…thonGH-29093)

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 <[email protected]>
bcail pushed a commit to bcail/cpython that referenced this pull request Apr 9, 2024
…thonGH-29093)

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 <[email protected]>
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