soreau / wf-kill

MIT License
2 stars 1 forks source link

Protocol not advertised #1

Closed UltraBlackLinux closed 2 years ago

UltraBlackLinux commented 2 years ago

Hey there, I just installed this plugin, and even after a reboot I seem to have trouble using it; I always get the following error: Wayfire kill view protocol not advertised by compositor. Is wf-kill plugin enabled? Why is that? I even tried rebooting, but to no avail.

Now, just out of curiosity, how is wf_kill_view_manager being set?

Thanks!

soreau commented 2 years ago

As the README states under Runtime, "Enable Wayfire Kill View Protocol plugin 'wf-kill'". You must put wf-kill in the list of [core] plugins or enable "Wayfire Kill View Protocol" in wcm>Utility category.

soreau commented 2 years ago

Now, just out of curiosity, how is wf_kill_view_manager being set?

Thanks!

The wf-kill client connects to the wayland display socket and sets up a registry handler, then calls roundtrip. This causes the compositor to call the registry handler for each advertised protocol, and wf-kill sets wf_kill_view_manager here.

UltraBlackLinux commented 2 years ago

As the README states under Runtime, "Enable Wayfire Kill View Protocol plugin 'wf-kill'". You must put wf-kill in the list of [core] plugins or enable "Wayfire Kill View Protocol" in wcm>Utility category.

The problem is that I did enable the plugin. Then I rebooted as it already didn't work.

soreau commented 2 years ago

The problem is that I did enable the plugin. Then I rebooted as it already didn't work.

That's strange, it should work.. can you upload the wayfire log somewhere and post the link here?

UltraBlackLinux commented 2 years ago

https://termbin.com/srvn

Oh I see: EE 08-07-22 09:04:38.510 - [src/output/plugin-loader.cpp:210] Failed to load plugin "wf-kill". Make sure it is installed in /usr/lib/wayfire or in $WAYFIRE_PLUGIN_PATH.

The installation instructions seem to install this into the wrong directory. It's not in /usr/lib/wayfire and the wayfire plugin path is unset

EDIT: Not even that I suppose. The build directory doesn't have any such so file. Just libwf-info.so

soreau commented 2 years ago

I see, I made a typo.. it's fixed now, please pull latest.

UltraBlackLinux commented 2 years ago

Works now, love it!

soreau commented 2 years ago

Awesome, glad it's working now. You can remove libwf-info.so or install wf-info to overwrite it.