https://bugzilla.redhat.com/show_bug.cgi?id=2421695
Yaakov Selkowitz <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] --- Comment #1 from Yaakov Selkowitz <[email protected]> --- Both the code (src/main.cpp) and the .desktop file use the `kwalletmanager` icon. Technically that means this needs to `Requires: kwalletmanager5` to provide the icon, but personally this seems like some sort of shortcut which they took in early development that they missed and they should just get their own icon instead. Could you please raise this upstream? URL should be https://apps.kde.org/keepsecret/ (no /fr/). Otherwise, the spec is straight-forward and per guidelines, and it builds and seems to work fine (tried it in a local flatpak build). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component https://bugzilla.redhat.com/show_bug.cgi?id=2421695 Report this comment as SPAM: https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=report-spam&short_desc=Report%20of%20Bug%202421695%23c1 -- _______________________________________________ package-review mailing list -- [email protected] To unsubscribe send an email to [email protected] Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/[email protected] Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
