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

Tk/Tcl upgrade #296

Open
apawlucz-cisco opened this issue Aug 8, 2024 · 8 comments
Open

Tk/Tcl upgrade #296

apawlucz-cisco opened this issue Aug 8, 2024 · 8 comments
Labels
dependencies Pull requests that update a dependency file

Comments

@apawlucz-cisco
Copy link

apawlucz-cisco commented Aug 8, 2024

Hi!

Thanks for the awesome work you do here, Gregory!

I have one ask though. Would it be possible to bump the tk/tcl version provided from 8.6.12 to 8.6.13 (or 8.6.14 which is the latest 8.6.x release)?

Last bump was with the release 20220222

That was bump 8.6.10 -> 8.6.12. 8.6.12 is almost 2-years old already.

Asking as we're experiencing an issue within our GUI app, which was fixed as part 8.6.13 release.

Thanks!

@apawlucz-cisco
Copy link
Author

apawlucz-cisco commented Aug 12, 2024

Just to clarify.

The issue we're currently facing with the latest PBS with tk/tcl 8.6.12 is causing the GUI dialog to not receive mouse events (something like 50% clicks or more are not received at all) on Mac. This issue was fixed in 8.6.13:

Mac: Mouse events not delivered to dialog with global grab.

https://sourceforge.net/p/tcl/mailman/message/37738603/

We managed to build python-build-standalone against tk/tcl 8.6.14 which indeed fixed the issue we observed.

We could post a PR for PSB if that helps.

Please let us know what you think.

@charliermarsh
Copy link
Member

I'm happy to review a PR if you want to send one in.

indygreg added a commit that referenced this issue Aug 24, 2024
Let's stay modern.

Should address #296.
zanieb pushed a commit to zanieb/python-build-standalone that referenced this issue Sep 4, 2024
Let's stay modern.

Should address astral-sh#296.
zanieb pushed a commit to zanieb/python-build-standalone that referenced this issue Sep 4, 2024
Let's stay modern.

Should address astral-sh#296.
zanieb pushed a commit to zanieb/python-build-standalone that referenced this issue Sep 4, 2024
Let's stay modern.

Should address astral-sh#296.
zanieb pushed a commit to zanieb/python-build-standalone that referenced this issue Sep 5, 2024
Let's stay modern.

Should address astral-sh#296.
@zanieb
Copy link
Member

zanieb commented Sep 7, 2024

Greg started in this in #303 and I'm following up in #313 — there's some non-trivial work to do still.

@apawlucz-cisco
Copy link
Author

Thanks a lot @zanieb!

@charliermarsh charliermarsh added the dependencies Pull requests that update a dependency file label Dec 18, 2024
zanieb pushed a commit to zanieb/python-build-standalone that referenced this issue Jan 1, 2025
Let's stay modern.

Should address astral-sh#296.
@apawlucz-cisco
Copy link
Author

Hi @zanieb!

Is there a chance for this to be fixed in a foreseeable future?

@zanieb
Copy link
Member

zanieb commented Jan 8, 2025

@apawlucz-cisco this is blocked by the Windows failure in the pull request — I don't know how to fix it yet.

@zanieb
Copy link
Member

zanieb commented Jan 8, 2025

We could split the update into Windows / non-Windows, perhaps.

@apawlucz-cisco
Copy link
Author

apawlucz-cisco commented Jan 9, 2025

I can only answer for myself here.

The tk issue we're experiencing doesn't happen on Windows. So for our project shipping updated tk only for non-Windows would solve the problem indeed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

No branches or pull requests

3 participants