Closed pepper-jelly closed 3 years ago
I can confirm this, I have a lot of these messages in my log, too.
So.... even with commit b9489850becd7977143b05a48d406c12ce83d7af, you get these messages, but only after you completely shut down Qtile, then start it up again?
On 201025 02:37, tcld wrote:
I can confirm this, I have a lot of these messages in my log, too.
-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/qtile/qtile/issues/1942#issuecomment-716119148
Yes, I am using that commit. The last time the message was logged after a simple restart, not a shutdown/restart of Qtile. I'll have to see if I can find a way to reproduce it, though, since it doesn't happen on every restart for me.
Note: The steps given by @leschalu above work to reproduce the issue. (shutdown+restart)
I wonder in that case if it's due to scratchpad windows staying in their
scratchpad state across restarts. The new qtile instance probably has the
scratchpads running as before, except because it wasn't this instance that
actually set up the scratchpad, no hooks were subscribed. This only applies to
dropdown windows whose on_focus_lost_hide
is True. Does it only happen when a
scratchpad dropdown is visible when the restart is triggered, and now when they
are all hidden on the scratchpad group?
On 201025 05:02, tcld wrote:
Yes, I am using that commit. The last time the message was logged after a simple restart, not a shutdown/restart of Qtile. I'll have to see if I can find a way to reproduce it, though, since it doesn't happen on every restart for me.
-- You are receiving this because you commented. Reply to this email directly or view it on GitHub: https://github.com/qtile/qtile/issues/1942#issuecomment-716136716
For me it happens even if the scratchpad isn't visible at the time of restart. I probably had to open it once before, though.
@m-col During session no errors which is good. The log will be empty if scratchpad window wasn't opened at all. To reproduce it has to be opened at least once. Session restart won't trigger it only shutdown(through lazy.shutdown or killall -u $USER). And it's doesn't matter if the terminal was hidden or not.
This happens as we subscribe to client_focus and setgroup only when on_focus_lost_hide is set but we unsubscribe uncoditionally. We have check if on_focus_lost_hide is set before trying to call unsubscribe.
@senthilbaboo are you on master (or at least after b9489850becd7977143b05a48d406c12ce83d7af)? That commit made the subscribe conditional upon on_focus_lost_hide
but despite that it seems the error message sometimes still appears.
Sorry about that. I just saw the thing that I was pointing out got fixed.
No probs! Although this is low priority, I'm stumped by it and not sure how to identify the problem :D
@leschalu @tcld do you get this on current master (aa66a3debc64557db39f79d2df8d44573be5227e)? I'm actually no longer able to reproduce, so maybe it got fixed as a side effect of a recent refactoring commit.
With latest master it still fills the same log but only when you logout and scratchpad window hidden. Doesn't matter if logout with killall -u $USER or lazy.shutdown()
Does https://github.com/qtile/qtile/pull/2112 fix it?
Does #2112 fix it?
yes looks good
Great, thanks.
Issue description
I opened previously #1936 I did test with latest commit at the moment with default config and added scratchpad group to the end.
Qtile version
commit 4e741da4b8126eb78fb9508e916cab350647f08e
Details
To reproduce - logout with
Ctrl Super q
(lazy.shutdown). And then log messages will appear on next login.