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

Sending the USR1 signal causes Xquartz to crash #29

Closed
matmunn opened this issue Jul 4, 2018 · 6 comments
Closed

Sending the USR1 signal causes Xquartz to crash #29

matmunn opened this issue Jul 4, 2018 · 6 comments
Assignees

Comments

@matmunn
Copy link

matmunn commented Jul 4, 2018

When sending conky the USR1 signal, which is meant to have conky reload the config file, causes Xquartz to crash, which then causes conky to crash and exit.

@npyl npyl self-assigned this Jul 5, 2018
@npyl
Copy link
Member

npyl commented Jul 5, 2018

With one quick look it seems to be working with the command killall -SIGUSR1 conky.
kill -s SIGUSR1 pid works, too!

@matmunn
Copy link
Author

matmunn commented Jul 5, 2018

I just had another look at this and interestingly doing kill -s SIGUSR1 pid worked, then kill -SIGUSR1 pid worked and restarted conky, then Xquartz crashed ~5s after the second signal was sent to conky.

@npyl
Copy link
Member

npyl commented Jul 5, 2018

I can confirm this myself, lets see what the problem is.

EDIT: The crash appears only with X11 support enabled. One of the two
(either conky or Xquartz) crashes though I am not sure which one.

Also, seems to me that the one crashing first is conky, then Xquartz.

@npyl
Copy link
Member

npyl commented Jul 8, 2018

Hi! Update on the problem: I think that disabling XDamage support makes the problem disappear. Please recompile with BUILD_XDAMAGE set to OFF

@matmunn
Copy link
Author

matmunn commented Jul 8, 2018

That seems to have fixed it for me. Thanks so much for working on this!

@matmunn matmunn closed this as completed Jul 8, 2018
@npyl
Copy link
Member

npyl commented Jul 8, 2018

No problem!!! Glad it helped!

npyl added a commit that referenced this issue Jul 9, 2018
We now disable XDamage support by default to allow SIGUSR1 to be handled properly without having Xquartz or conky crash.
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

No branches or pull requests

2 participants