FS#75648 - [mattermost-desktop] Missing gnome shortcut

Attached to Project: Community Packages
Opened by Nicolas Repentin (ouafnico) - Thursday, 18 August 2022, 21:01 GMT
Last edited by T.J. Townsend (blakkheim) - Friday, 15 September 2023, 17:01 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Caleb Maclennan (alerque)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 2
Private No

Details

Description:
After installing the package there is no shortcut on gnome interface.

You can start the app calling manually mattermost-desktop.

Additional info:
* package version(s) 5.1.1-3
This task depends upon

Closed by  T.J. Townsend (blakkheim)
Friday, 15 September 2023, 17:01 GMT
Reason for closing:  Fixed
Additional comments about closing:  5.4.0-4
Comment by Toolybird (Toolybird) - Thursday, 18 August 2022, 22:47 GMT
usr/share/applications/mattermost-desktop.desktop

definitely exists in the pkg so something is amiss on your system. Rebooted?
Comment by Nicolas Repentin (ouafnico) - Friday, 19 August 2022, 12:39 GMT
Hum indeed I got the file, but on two differents machines I got the same behavior.
I've rebooted too.

Will check if there is some syntax problem on the desktop file.
Comment by Ghislain Bourgeois (ghibourg) - Saturday, 03 September 2022, 01:42 GMT
I had the same problem, there were two errors in the desktop file. The lines for Categories and MimeType contained an extra `;` at the end of the line. I copied the file to ~/.local/share/applications/ and fixed it with this content:

[Desktop Entry]
Version=1.0
Type=Application
Name=Mattermost
Comment=Mattermost Desktop application
Exec=/usr/bin/mattermost-desktop
Icon=mattermost-desktop
Terminal=false
StartupNotify=false
Categories=Network;InstantMessaging;
MimeType=x-scheme-handler/mattermost;
StartupWMClass=mattermost-desktop

I then executed `update-desktop-database .local/share/applications/` and the shortcut worked.
Comment by Toolybird (Toolybird) - Sunday, 04 September 2022, 00:23 GMT
@ghibourg, well spotted! We supply that file so there is some kind of packaging bug here...
Comment by Yohan Boniface (ybon) - Tuesday, 01 November 2022, 10:24 GMT
Same issue for me. I've used the workaround by @ghibourg, and it worked for me too. Thanks :)
Comment by Sebastian Wiesner (lunaryorn) - Tuesday, 01 August 2023, 07:42 GMT
The attached patch fixes the desktop file and removes the unused separate file from the package. I've mailed it to the package maintainers a few weeks ago, but haven't received a response yet.
   0001-Fix-desktop-file-for-mat... (2 KiB)
Comment by Sebastian Wiesner (lunaryorn) - Monday, 11 September 2023, 15:49 GMT
The desktop entry also has other flaws; in particular it also specifies the wrong wmclass. I've attached another patch which creates a working desktop entry.

Given that this bug is open for over a year, even though the fix is trivial, I've kinda given up though, and have started building this package myself. If this package is no longer maintained, perhaps you could drop it to the AUR, so that the community can take over and fix it?
   mattermost-desktop.patch (2.6 KiB)
Comment by T.J. Townsend (blakkheim) - Friday, 15 September 2023, 17:01 GMT
It is discouraging to submit patches and get no response for a long time. I know the feeling.

Loading...