Bug 233591 - mail/thunderbird has incorrect icon name in desktop file
Summary: mail/thunderbird has incorrect icon name in desktop file
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Mateusz Piotrowski
Depends on:
Reported: 2018-11-27 23:46 UTC by Zach Dykstra
Modified: 2020-01-31 17:00 UTC (History)
3 users (show)

See Also:
bugzilla: maintainer-feedback? (gecko)


Note You need to log in before you can comment on or make changes to this bug.
Description Zach Dykstra 2018-11-27 23:46:06 UTC
The thunderbird.desktop file has an incorrectly defined Icon entry. Icon entries with out an explicit full path to the icon should be set with out a file suffix.


This should instead be:


With it set to thunderbird.png, launchers and other .desktop consumers are looking for thunderbird.png.{png,xpm,svg}, which will fail. Removing the extension allows 'rofi' to correctly show an icon for Thunderbird in the launcher.
Comment 1 Zach Dykstra 2018-11-28 00:10:12 UTC
--- Makefile    2018-11-27 18:09:01.939635000 -0600
+++ Makefile.no-suffix  2018-11-27 18:08:39.801660000 -0600
@@ -58,7 +58,7 @@
-       @${SED} -e 's|@PORTNAME_ICON@|${PORTNAME_ICON}|;s|@MOZILLA@|${MOZILLA}|' \
+       @${SED} -e 's|@PORTNAME_ICON@|${PORTNAME_ICON:R}|;s|@MOZILLA@|${MOZILLA}|' \
                <${FILESDIR}/thunderbird.desktop.in >${WRKDIR}/${MOZILLA_EXEC_NAME}.desktop

The above diff seems to generate the correct work/thunderbird.desktop file.
Comment 2 Walter Schwarzenfeld freebsd_triage 2019-08-02 11:53:30 UTC
Comment 3 Mateusz Piotrowski freebsd_committer 2019-10-27 09:57:15 UTC
I've tested the patch. It seems to work. I'll try to get it committed soon.

Thanks a lot for your contribution!
Comment 4 Mateusz Piotrowski freebsd_committer 2019-10-27 11:00:50 UTC
Comment 5 commit-hook freebsd_committer 2019-10-29 09:02:34 UTC
A commit references this bug:

Author: 0mp
Date: Tue Oct 29 09:02:03 UTC 2019
New revision: 515968
URL: https://svnweb.freebsd.org/changeset/ports/515968

  Change icon entry in Thunderbird desktop file

  By setting the "Icon" entry to "thunderbird" from "thunderbird.png" we
  allow programs like rofi to detect the Thunderbird icon.

  PR:		233591
  Submitted by:	Zach Dykstra <soporose@gmail.com>
  Approved by:	portmgr (blanket: run bugfixes)
  MFH:		2019Q4
  Differential Revision:	https://reviews.freebsd.org/D22167