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 169202 - Problem with "Open with other application"-created .desktop files
Problem with "Open with other application"-created .desktop files
Status: RESOLVED FIXED
Product: eel
Classification: Deprecated
Component: general
2.13.x
Other Linux
: High major
: ---
Assigned To: Nautilus Maintainers
Nautilus Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-03-04 15:27 UTC by Christian Persch
Modified: 2006-02-07 10:14 UTC
See Also:
GNOME target: ---
GNOME version: 2.13/2.14


Attachments
proposed patch (1.17 KB, patch)
2005-03-04 15:28 UTC, Christian Persch
needs-work Details | Review
just the .desktop filename changes part of the patch (981 bytes, patch)
2005-03-05 10:50 UTC, Christian Persch
committed Details | Review
just the Exec= part, with %f instead of %U (558 bytes, patch)
2005-03-05 12:12 UTC, Christian Persch
none Details | Review

Description Christian Persch 2005-03-04 15:27:32 UTC
Their name can clash with system .desktop files, causing them to have precedence
over the system one (see bug 169198); and their Exec line makes them unsuited
for launching with gnome_vfs_mime_application_launch and gnome_desktop_item_launch.
Comment 1 Christian Persch 2005-03-04 15:28:03 UTC
Created attachment 38250 [details] [review]
proposed patch
Comment 2 Marco Pesenti Gritti 2005-03-04 16:29:44 UTC
Hmm gnome-vfs should be able to deal with Exec without %U for compatibility
reasons. I cannot reproduce this in rawhide, maybe it's something changed
recently...

At least, if add gedit --new-window with nautilus and then launch it, the file
is opened correctly.
Comment 3 Christian Persch 2005-03-05 10:50:33 UTC
Created attachment 38290 [details] [review]
just the .desktop filename changes part of the patch
Comment 4 Marco Pesenti Gritti 2005-03-05 11:59:39 UTC
Comment on attachment 38250 [details] [review]
proposed patch

If you could split this and I think we should use %f, not %U.
Comment 5 Christian Persch 2005-03-05 12:12:35 UTC
Created attachment 38295 [details] [review]
just the Exec= part, with %f instead of %U
Comment 6 Sebastien Bacher 2005-05-08 12:15:27 UTC
the patch is on eel, reassigning to the right component.
Comment 7 Sebastien Bacher 2006-01-08 22:39:01 UTC
The first patch has been commited by alex:

2005-03-07  Alexander Larsson  <alexl@redhat.com>

        * eel/eel-mime-extensions.c: (eel_mime_add_application):
        Make sure user added desktop files don't conflict with global ones.
        Patch from Christian Persch


Christian, could you mail the second patch on the nautilus-list so a maintainer can accept it?
Comment 8 Alexander Larsson 2006-02-07 10:14:29 UTC
Commited the second patch. Thanks.