After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 654707 - [PATCH] favorite-apps: Use firefox.desktop & libreoffice-writer.desktop
[PATCH] favorite-apps: Use firefox.desktop & libreoffice-writer.desktop
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
unspecified
Other Linux
: Normal minor
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2011-07-15 22:17 UTC by Jeremy Bicha
Modified: 2011-08-22 18:41 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
updated-favorite-apps (1.15 KB, patch)
2011-07-15 22:24 UTC, Jeremy Bicha
reviewed Details | Review
0002-favorite-apps-Use-libreoffice-writer-firefox.desktop.patch (1.32 KB, patch)
2011-07-29 14:03 UTC, Jeremy Bicha
committed Details | Review

Description Jeremy Bicha 2011-07-15 22:17:42 UTC
At least Debian, OpenSUSE & Ubuntu ship firefox.desktop instead of mozilla-firefox.desktop. Also Linux distributions have effectively chosen LibreOffice over OpenOffice.org.
Comment 1 Jeremy Bicha 2011-07-15 22:24:39 UTC
Created attachment 192061 [details] [review]
updated-favorite-apps
Comment 2 Matthias Clasen 2011-07-25 11:29:19 UTC
The libreoffice change is certainly right.
For firefox, maybe we can push the fedora packagers to switch to (or add) a firefox.desktop
Comment 3 Jeremy Bicha 2011-07-26 02:13:42 UTC
Well, Fedora's source RPM already has firefox.desktop. I think it's just their firefox.spec which installs it as mozilla-firefox.desktop instead so it should be a very easy fix for the Fedora packagers.
Comment 4 drago01 2011-07-29 13:42:02 UTC
Review of attachment 192061 [details] [review]:

Looks fine, and re fedora we got the same vendor prefix mess with gnome-nautilus.desktop vs. nautilus.desktop and just fixed it anyway.

The patch should contain a commit message though; and put a link to the bug at the bottom of rather then (#bugnr) in the subject.
Comment 5 Jeremy Bicha 2011-07-29 14:03:04 UTC
Created attachment 192877 [details] [review]
0002-favorite-apps-Use-libreoffice-writer-firefox.desktop.patch

Thanks for the review. Here's an improved version.
Comment 6 Jasper St. Pierre (not reading bugmail) 2011-08-03 19:12:16 UTC
If you can submit the libreoffice change as a separate patch, we could land that, and then we'd just be waiting for F15 to change the Firefox desktop filename.
Comment 7 Jeremy Bicha 2011-08-03 19:39:14 UTC
Why can't GNOME make a change without consulting Fedora first? Isn't it the distro packagers' job to package what upstream creates? Fedora 15 doesn't use Gnome Shell 3.1.* so why does Fedora 15 need to change now instead of waiting for Fedora 16? 

As I mentioned in comment 3, they actually already have firefox.desktop in their package; they just told their .spec to ship it as mozilla-firefox.desktop instead. It should just be a one line change in Fedora 16 as Gnome Shell 3.1.5 is released if Fedora were doing what I would have expected. But it sounds like Fedora intends to support Gnome Shell 3.0.* and 3.1.* and the bleeding-edge git snapshot in their supposedly stable release. So it would additionally require a Gnome Shell 3.0 patch.

If Fedora wants to act different than every other distribution, why should that slow down Gnome Shell doing what's best? And can't they still make the change immediately after?
Comment 8 Jeremy Bicha 2011-08-03 19:41:05 UTC
Or I guess they could do a symlink or something instead of patching Gnome Shell but still.
Comment 9 Jeremy Bicha 2011-08-20 07:16:28 UTC
I don't have commit access and I don't intend to split a perfectly good one line patch into two parts for the wrong reason. I don't care if my name shows up in the git log for this; I'd just like someone to fix this bug.

The LibreOffice part is broken with Fedora right now and has been for months. The Firefox part has to be patched by several distros (including Fedora the way I read the firefox .spec).
Comment 10 Colin Walters 2011-08-22 13:53:27 UTC
Review of attachment 192877 [details] [review]:

Looks good to me, thank you!
Comment 11 Matthias Clasen 2011-08-22 13:54:07 UTC
> The Firefox part has to be patched by several distros (including Fedora the way
> I read the firefox .spec).

That is part of the problem here. This is not about giving Fedora some special role, it is about getting things fixed at the source... we are pushing to get mozilla to include a proper desktop file. 

But we should get this change committed regardless.
Comment 12 Jasper St. Pierre (not reading bugmail) 2011-08-22 13:57:08 UTC
Thanks!
Comment 13 Jeremy Bicha 2011-08-22 18:41:10 UTC
Thanks!