Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add support for the wayland platform #540

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

Open
buresu wants to merge 1 commit into probonopd:master
base: master
Choose a base branch
Loading
from buresu:wayland-platform-support

Conversation

Copy link

@buresu buresu commented Jul 12, 2022
edited
Loading

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

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

close #189

JLP and sonic2kk reacted with thumbs up emoji
@buresu buresu marked this pull request as draft July 12, 2022 10:07
@buresu buresu force-pushed the wayland-platform-support branch from d77f81a to 2140d90 Compare July 12, 2022 11:56
@buresu buresu marked this pull request as ready for review July 12, 2022 11:57
Copy link

ardera commented Jan 18, 2023

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

Copy link
Owner

probonopd commented Jan 18, 2023
edited
Loading

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.

ali1234 reacted with thumbs up emoji tulpenkiste, intrnl, 1peter10, Vankata453, junglerobba, jibsaram, rumpelsepp, JLP, henriqueffc, networkException, and 28 more reacted with thumbs down emoji cassidyjames reacted with laugh emoji tulpenkiste, Vankata453, jibsaram, rumpelsepp, asjur, Darth-Tiktaalik, Fxsch, and IvarWithoutBones reacted with confused emoji asjur reacted with eyes emoji

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.

Vankata453, maya-doshi, Be-ing, stgraber, Sprinter05, Darth-Tiktaalik, Fxsch, vlinkz, edfloreshz, banjoman05, and 13 more reacted with thumbs up emoji

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.

tulpenkiste, colinkiama, TheSandBoxMKG, xerz-one, spazzylemons, gikari, CarlSchwan, sonic2kk, and Nutjob reacted with thumbs up emoji

Copy link
Owner

probonopd commented Jan 20, 2023
edited
Loading

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
Copy link
Owner

probonopd commented Feb 20, 2023
edited
Loading

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.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Deploy the Wayland platform plugin

AltStyle によって変換されたページ (->オリジナル) /