Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 832739 - sys-apps/lshw[gtk]: invalid icon in desktop entry
Summary: sys-apps/lshw[gtk]: invalid icon in desktop entry
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Ben Kohler
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2022-02-05 07:21 UTC by i.Dark_Templar
Modified: 2022-02-06 12:59 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
lshw-desktop.patch (lshw-desktop.patch,533 bytes, patch)
2022-02-05 07:23 UTC, i.Dark_Templar
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description i.Dark_Templar 2022-02-05 07:21:08 UTC
I've noticed that 'Hardware Lister' menu entry in KDE doesn't have an icon, it only has placeholder icon. On further inspection I've noticed that desktop entry lists 'lshw' as icon, while this package packs only gtk-lshw.svg and no lshw.svg.

Reproducible: Always

Steps to Reproduce:
1. USE="gtk" emerge -1 sys-apps/lshw
2. run DE if not running yet, KDE for example
3. Locate 'Hardware Lister' menu entry in main menu. For me it's under 'System' menu group, i.e. group containing "konsole" and "dolphin" menu entries, for example.
Actual Results:  
'Hardware Lister' menu entry has placeholder icon

Expected Results:  
'Hardware Lister' menu entry should have proper icon

$ equery f lshw | grep icons
/usr/share/icons
/usr/share/icons/hicolor
/usr/share/icons/hicolor/scalable
/usr/share/icons/hicolor/scalable/apps
/usr/share/icons/hicolor/scalable/apps/gtk-lshw.svg

$ cat /usr/share/applications/gtk-lshw-lshw.desktop 
[Desktop Entry]
Name=Hardware Lister
Type=Application
Comment=Hardware Lister
Exec=/usr/sbin/gtk-lshw
TryExec=/usr/sbin/gtk-lshw
Icon=lshw
Categories=System;
Comment 1 i.Dark_Templar 2022-02-05 07:23:56 UTC
Created attachment 764369 [details, diff]
lshw-desktop.patch

Patch file containing one way of fixing this issue. Either icon should be renamed to match what desktop file contains, or desktop file should be updated to reflect real icon name. This patch renames icon.
Comment 2 Ben Kohler gentoo-dev 2022-02-06 12:54:46 UTC
Thanks, since we're already renaming with newicon, I agree that just renaming properly is the best fix
Comment 3 Larry the Git Cow gentoo-dev 2022-02-06 12:59:03 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=a69cd51a1b046bf146aafc1c667638f40289db94

commit a69cd51a1b046bf146aafc1c667638f40289db94
Author:     Ben Kohler <bkohler@gentoo.org>
AuthorDate: 2022-02-06 12:58:45 +0000
Commit:     Ben Kohler <bkohler@gentoo.org>
CommitDate: 2022-02-06 12:58:45 +0000

    sys-apps/lshw: revbump for desktop file icon fix
    
    Closes: https://bugs.gentoo.org/832739
    Package-Manager: Portage-3.0.30, Repoman-3.0.3
    Signed-off-by: Ben Kohler <bkohler@gentoo.org>

 sys-apps/lshw/lshw-02.19.2b_p20210121-r3.ebuild | 83 +++++++++++++++++++++++++
 1 file changed, 83 insertions(+)