-
Notifications
You must be signed in to change notification settings - Fork 258
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
bundle qt platform plugins #602
Conversation
138a9f5
to
28b4c0d
Compare
Are there some more platforms that should be bundled?
I guess vnc might be an option |
f3fbb6b
to
a7aba6f
Compare
I tested the resulting appimage under Debian 11, 12, ubuntu 22.10, 23.10, centos7 and it works |
If VNC is an option and does not increase the size much, then that would indeed be nice. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you please expand on what is experimental about wayland? did you test it on a wayland environment?
what value does that bring? can't we rely on XWayland compatibility?
I disagree. the appimage serves a general purpose, but providing all bells and whistles should not be the target. bloating the size "because we can" is not a good goal. if you want to remote something, just do that yourself |
That all depends - if "we can easily" and there's an increase of 65.5 to 66.5 (that would be less than 2%), then this would be nice. If it is an increase by 20% or is complicated, then that is bad. |
Adding wayland support provides proper scaling. I know there are some environment variables that you can use but these mess with other programs (e.g. zoom) so you need a lens to use these programs. |
but why is this "experimental"? |
Because the appimage guys don't support it. But that is expected of a wayland hater. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ping? this could probably go in with some more explanations and fixese
a7aba6f
to
180ad9a
Compare
this gives the user more leeway to use the appimage closes: #554
180ad9a
to
46e484c
Compare
Just out of interest: I've got a machine where the CI artifact wants to start, but aborts because of a missing library - should this be installed or was it a bad link? ./hotspot-v1.5.0-17-gd2703f3-x86_64.AppImage --help --> missing library outside of QT starting a GUI process; would it be possible/reasonable to bundle it as well? |
Hm, rechecking: maybe #282 should be reopened.
with
So the libKF5* libraries have a dependency to other libraries like libQt5Gui.so.5 (that are included in the appimage) which have the dependency to libGL.so.1, which is not part of the appimage. I think that with the current infrastructure libGL.so.1 should be included as part of this PR. Ideally (but that would be a separate issue) the libKF5* libraries would be delay-loaded, this would allow |
this gives the user more leeway to use the appimage closes: 554