-
Notifications
You must be signed in to change notification settings - Fork 49
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
Disable crashing calls on Wayland #469
Conversation
They are screen locking before suspend and reloading the settings of `lxqt-config-monitor` Fixes #457 and is related to lxqt/lxqt-config#916
There's something: on suspend/resume
Session doesn't crash, and |
Also lxqt/lxqt-powermanagement#342 is present again, it was solved in the previous PR. |
This PR includes the previous one; it just adds Are you sure that you've compiled/tested the correct branch? |
Yes. But recompiled to be sure and now the crash is gone. But didn't switch branches... |
ok, it looks like the crash happens only if "lock screen on lid close" is checked in |
I don't think it was. More probably, you've found another source of crash on Wayland. |
What is strange: I change settings from in "lid action" from "suspend" to "lock screen" or vice-versa and it activates the new setting only at the 2nd lid closure. Will do more tests. Btw "Ask" is still there as an option for "lid close action". |
Oh, the last crash is the same as in your other report: lxqt/lxqt-powermanagement#342. It isn't related to lxqt-session. |
Yes. I tested again #468 and it looks that I was wrong about lxqt/lxqt-powermanagement#342 : it was not solved there too. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Session crash at suspend is prevented and no records of coredumps for lxqt-config-monitor
too.
They are screen locking before suspend and reloading the settings of
lxqt-config-monitor
Fixes #457 and is related to lxqt/lxqt-config#916