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

AppImage: don't package libwayland-client.so.0 #24111

Conversation

cbjeukendrup
Copy link
Contributor

@cbjeukendrup cbjeukendrup commented Aug 20, 2024

Resolves: #24068

@cbjeukendrup cbjeukendrup marked this pull request as ready for review August 21, 2024 00:36
@cbjeukendrup cbjeukendrup requested a review from shoogle August 21, 2024 00:36
@kbloom kbloom mentioned this pull request Aug 21, 2024
8 tasks
@@ -208,7 +208,8 @@ function fallback_library()
# Report new additions at https://github.com/linuxdeploy/linuxdeploy/issues
# or https://github.com/linuxdeploy/linuxdeploy-plugin-qt/issues for Qt libs.
unwanted_files=(
# none
# https://github.com/musescore/MuseScore/issues/24068#issuecomment-2297823192
lib/libwayland-client.so.0
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It might also be worth mentioning libwayland-client.so.0 (without lib/ prefix) in the fallback_libraries array further down. This way it will be loaded if it's not found on the system, which could prevent a crash.

You'll need to keep it here in unwanted_files (with lib/ prefix) so that it doesn't also get packaged in the lib folder as well as the fallback folder.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Making it fallback is potentially only worth doing on the 4.4 branch, whereas excluding it is worth doing on both branches.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll add it to fallback. Why would that not necessary on master too?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't know for sure that excluding it will cause a crash, so I want to find out one way or the other and then report the outcome to the Linuxdeploy developers.

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

Successfully merging this pull request may close these issues.

4.4 Beta AppImage won't start on Manjaro Linux on wayland
2 participants