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

debian bug 1034054: backport fvwm3 xthreadlock() #102

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ThomasAdam
Copy link
Member

Backport this fix from fvwm3 to fvwm2:

fvwmorg/fvwm3@5c17c83

@ThomasAdam ThomasAdam self-assigned this May 2, 2023
@hills
Copy link

hills commented Jan 29, 2024

I've been running this patch for nearly 1 week now and it has fixed the related crashes, so I recommend it for merge into the 2.x series as a priority. An update to libX11 means a regression in FVWM behaviour and this patch prevents it.

@somiaj
Copy link
Contributor

somiaj commented Jan 29, 2024

There is a minor issue with this patch, that although fvwm no longer crashes (woot), some expose events are lost and icons do not refresh correctly. See fvwmorg/fvwm3#818, which hasn't been fully resolved, though there is some ideas of how to fix that issue.

Note the icon issue is much smaller than the crashing issue, so probably best to at least not crash. This patch is included in Debian's package, though fvwm is basically not supported, so it even if this gets merged, a new release might not be made. In either case users should grab this patch to avoid the libx11 change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants