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

Fix for missing icons: Require Qt SVG plug-in #60

Merged
merged 2 commits into from
Nov 8, 2018
Merged

Fix for missing icons: Require Qt SVG plug-in #60

merged 2 commits into from
Nov 8, 2018

Conversation

shundhammer
Copy link
Contributor

Bugzilla

https://bugzilla.suse.com/show_bug.cgi?id=1114654

Trello

https://trello.com/c/27ozQgUS/452-ostumbleweed-p1-1114654-build-20181102-patternselector-icons-missing-openqa-test-fails-in-selectpatternsandpackages

Problem

No status icons in the patterns or package selector in Qt, in the inst-sys as well as in the installed system later.

Cause

No support for SVG available. We have compiled-in fallback icons, but they are also SVG now, so they cannot be rendered or maybe even not loaded.

In previous versions, the libyui-qt-pkg plugin was linked against libqt5svg.so (it was listed in the ldd) output, but since this current Tumbleweed snapshot it is no longer. Why this is so is not completely clear; very likely it was an indirect dependency where some other part of Qt was using it. Even trying to force linking it to libyui-qt-pkg failed every time; we are not actively using it, only indirectly, and it is a plug-in to the Qt image loading subsystem; so the linker saw our link instruction and still decided that it is not necessary and simply did not link that lib to our binary (I tried different approaches with that).

Solution

Add a plain and simple explicit Requires for the package (not the library) to our .spec file.

In the inst-sys

@wfeldt said if we have that dependency on our package it will automatically be honored for building the inst-sys.

Copy link
Member

@lslezak lslezak left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@shundhammer shundhammer merged commit bcc6e5b into libyui:master Nov 8, 2018
@shundhammer shundhammer deleted the huha-missing-icons-02 branch November 8, 2018 11:02
@shundhammer
Copy link
Contributor Author

mvidner pushed a commit to mvidner/libyui-combined-alpha-3 that referenced this pull request Jan 15, 2021
…ing-icons-02

Fix for missing icons: Require Qt SVG plug-in
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.

None yet

2 participants