Conversation
|
I see this in log: |
There was a problem hiding this comment.
$wm_name is defined here. Are we sure that the proposed patch triggers the issue? It looks like a new issue to me, originally the whole $wnck_screen was undefined, now it just fails to get the window manager's name. Maybe we can catch it and set $wm_name to something like "not-compiz", if $wnck_screen->get_window_manager_name returns undefined?
|
@DarthGandalf Could you please test if the latest commit fixes the issue for you? |
|
| @@ -8455,6 +8470,8 @@ sub STARTUP { | |||
| my $wm_name; | |||
| if ($wnck_screen->can('get_window_manager_name')) { | |||
| $wm_name = $wnck_screen->get_window_manager_name; | |||
There was a problem hiding this comment.
So, basically, $wnck_screen->can('get_window_manager_name') returns true, but $wnck_screen->get_window_manager_name is empty...
|
@DarthGandalf Now that 0.99.7 is out, there is no hurry, but if you have some spare time, please test the latest commit, I think, it should fix the issue. 😃 |
See https://gitlab.gnome.org/GNOME/libwnck/-/issues/162#note_2338372