drowe67 / freedv-gui

GUI Application for FreeDV – open source digital voice for HF radio
https://freedv.org/
GNU Lesser General Public License v2.1
193 stars 49 forks source link

FreeDV icon does not appear for Linux version #740

Closed tmiw closed 3 weeks ago

tmiw commented 1 month ago

For example, on GNOME:

image

whereas on macOS:

image

(and similarly on Windows)

Anyway, the icon that's displayed should be the same regardless of platform. wxWidgets' SetIcon() might be usable for this, or perhaps a .desktop file needs to be created for FreeDV and installed in the correct location when running make install.

Tyrbiter commented 4 weeks ago

This is the output of rpm -ql for the Fedora freedv rpm:

/usr/bin/freedv /usr/lib/.build-id /usr/lib/.build-id/6b /usr/lib/.build-id/6b/18bb958be26c1c4c8e763d1694e46a7b7fdf39 /usr/share/appdata/freedv.appdata.xml /usr/share/applications/freedv.desktop /usr/share/doc/freedv /usr/share/doc/freedv/README.md /usr/share/doc/freedv/USER_MANUAL.md /usr/share/freedv-gui/wav /usr/share/freedv-gui/wav/all_2020.wav /usr/share/freedv-gui/wav/ve9qrp_1600.wav /usr/share/freedv-gui/wav/ve9qrp_700c.wav /usr/share/freedv-gui/wav/ve9qrp_700d.wav /usr/share/freedv-gui/wav/ve9qrp_700e.wav /usr/share/freedv-gui/wav/ve9qrp_800xa.wav /usr/share/icons/hicolor/128x128/apps/freedv.png /usr/share/icons/hicolor/256x256/apps/freedv.png /usr/share/icons/hicolor/48x48/apps/freedv.png /usr/share/icons/hicolor/64x64/apps/freedv.png /usr/share/licenses/freedv /usr/share/licenses/freedv/COPYING

As you can see there are various freedv.png files in different resolutions and also a .desktop file

This should be the same in the current distro version which is freedv-1.8.4 so the files needed are readily accessible.

tmiw commented 4 weeks ago

Sounds like it might be something the various package maintainers are already doing, then? If that's the case, do we need to take action on this at all?

Tyrbiter commented 4 weeks ago

Sounds like it might be something the various package maintainers are already doing, then? If that's the case, do we need to take action on this at all?

I would say that as long as there is a suitable icon image file somewhere it's something a packager ought to do, @barjac is the Mageia packager so I would expect it to be in there somewhere but I have not checked what his packages contain.

barjac commented 4 weeks ago

Yes I do the same in Mageia, we have the various size icons and these are used for the .desktop file and also show on the taskbar along with the name and also in the main application menu.

Screenshot_20240814_202120

I would say that as long as there is a suitable icon image file somewhere it's something a packager ought to do

I agree this is something that packagers should do so it's not an issue.

tmiw commented 3 weeks ago

I agree this is something that packagers should do so it's not an issue.

Fair enough, I'll go ahead and close for now.