Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for the wayland platform #540

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

buresu
Copy link

@buresu buresu commented Jul 12, 2022

The following library and plugins are required to support the wayland platform.

libQt5WaylandClient.so
libQt5WaylandCompositor.so
platforms/libqwayland-*
wayland-*

close #189

@buresu buresu marked this pull request as draft July 12, 2022 10:07
@buresu buresu marked this pull request as ready for review July 12, 2022 11:57
@ardera
Copy link

ardera commented Jan 18, 2023

@probonopd Friendly ping, do you have any time to look at this? :)

@probonopd
Copy link
Owner

probonopd commented Jan 18, 2023

Thanks, but I am not interested in Wayland. None of my systems run Wayland, because it never works quite right in my experience. Especially when you are not using GNOME or KDE.

@probonopd probonopd closed this Jan 18, 2023
@tulpenkiste
Copy link

Thanks, but I am not interested in Wayland. None of my systems run Wayland, because it never works quite right in my experience. Especially when you are not using GNOME or KDE.

Hi probonopd,
I do not understand the reasoning you are using here to continue ignoring Wayland support on your software.

When it comes to not being interested in Wayland, that is fine. But some people (like myself) use Wayland on a daily basis (even using it exclusively).

However, this PR that you've rejected would have not affected you in the slightest as XOrg support would still be usable. It would have just improved the experience on Wayland as we wouldn't need to rely on xwayland to even be able to run it.

Plus, when it comes to Wayland "not working right", I can say the exact same about XOrg as it had always been janky and unstable for me (including but not limited to random crashes, random screen tearing and widgets acting weirdly in KDE Plasma) which I cannot say the same about Wayland. Even then, if I do make an application that uses a graphical interface, I won't just skip XOrg support because that would be quite odd. So why should it be the same for Wayland support, especially when the PR gave you literally all the information you needed to implement it?

These kinds of decisions by you on AppImage related projects makes me consider things like AppImages last when going to obtain new software. Right behind compiling software from source and Flatpak.

@mothdotmonster
Copy link

I think that it's a good idea to add support, as part of the reason why Wayland isn't quite ready for prime time is because not everything supports it yet! So adding extra support to things will make Wayland better.

@probonopd
Copy link
Owner

probonopd commented Jan 20, 2023

So adding extra support to things will make Wayland better.

Thanks, but I am not interested in making Wayland better. I think that is the job of the people who are pushing Wayland - to make it run existing X11 applications flawless without requiring changes in the applications.

It has been claimed that Wayland can run X11 applications using XWayland. So it should be able to run "normal" applications just fine?

Repository owner locked as resolved and limited conversation to collaborators Jan 20, 2023
@probonopd probonopd reopened this Feb 20, 2023
@probonopd
Copy link
Owner

probonopd commented Feb 20, 2023

Code change looks reasonable; however we need to find out whether this causes any issues for non-Wayland users, and whether this draws in additional dependencies which increase the AppImage size significantly. Any other potential risks/downsides?

Since I don't run Wayland I can't test the Wayland side of this.

Remains the philosophical question:
Should we really help promoting the splintering of the *nix desktop by shipping Wayland-specific stuff? I tend to think that the better way is to make AppImages use X11 (since that is what most systems use) and have Linux distributions who insist on already shipping Wayland ensure that Wayland can run X11 applications very, very well.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deploy the Wayland platform plugin
5 participants