Check that Xwayland exists before enabling it

[earboxer]

Describe your PR, what does it fix/add?

Fixes launching Hyprland when you don't have Xwayland installed

Hyprland 0.46 would crash almost immediately if Xwayland wasn't installed unless you put xwayland:enabled = false in your config.

Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)

Is it ready for merging, or does it need work?

I think this is ready for merging

[littleblack111]

wouldn't it be better for it to be set during compile?

[vaxerski]

wouldnt it be better for this check to happen in the ctor of cxwayland so that it skips server

[vaxerski]

@littleblack111 no

[earboxer]

wouldnt it be better for this check to happen in the ctor of cxwayland so that it skips server

Can any of Xwayland work if it's not installed?

I was thinking with putting it here that if it wasn't installed, one could install it, then reload the config and hyprland would initialize xwayland, seeing that prevEnabledXwayland is not the same as PENABLEXWAYLAND.

[vaxerski]

I was talking about moving the executable check to the ctor of CXWayland. By the way we could also move m_bEnableXWayland -> CXWayland::enabled() (which is based on the ctor arg const bool enabled)

[earboxer]

@vaxerski Good point. Refactored.

I also made it so it doesn't disable xwayland if it was already running and you wanted it enabled but uninstalled it.