Javyre / swayfire

Sway/I3 inspired tiling window manager for Wayfire
GNU General Public License v3.0
172 stars 8 forks source link

Fails to build with latest Wayfire #41

Closed jbeich closed 3 years ago

jbeich commented 3 years ago

Regressed by https://github.com/wayfirewm/wayfire/commit/a79978b20b0d.

In file included from src/core/binding.cpp:2:
src/core/core.hpp:1182:34: error: no type named 'view_change_viewport_signal' in namespace 'wf'; did you mean 'view_change_workspace_signal'?
                dynamic_cast<wf::view_change_viewport_signal *>(data_);
                             ~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~
                                 view_change_workspace_signal
/usr/include/wayfire/signal-definitions.hpp:604:8: note: 'view_change_workspace_signal' declared here
struct view_change_workspace_signal : public _view_signal
       ^
Javyre commented 3 years ago

Yes sorry I was in a hurry and forgot to update https://github.com/Javyre/swayfire/pull/40 before merging https://github.com/Javyre/swayfire/commit/a5c38cf50659665cd98599392a993731b13fa8aa should fix the build.

jbeich commented 3 years ago

I confirm, builds fine now.