[prev in list] [next in list] [prev in thread] [next in thread]
List: freedesktop-portland-bugs
Subject: [Portland-bugs] [Bug 26300] New: [xdg-settings] [PATCH] Should use
From: bugzilla-daemon () freedesktop ! org
Date: 2010-01-28 17:33:28
Message-ID: bug-26300-6947 () http ! bugs ! freedesktop ! org/
[Download RAW message or body]
http://bugs.freedesktop.org/show_bug.cgi?id=26300
Summary: [xdg-settings] [PATCH] Should use basenames in GNOME
Product: Portland
Version: unspecified
Platform: Other
OS/Version: All
Status: NEW
Severity: normal
Priority: medium
Component: xdg-utils
AssignedTo: portland-bugs@lists.freedesktop.org
ReportedBy: michael.terry@canonical.com
Created an attachment (id=32875)
--> (http://bugs.freedesktop.org/attachment.cgi?id=32875)
Use basenames for GNOME
In GNOME, it's more appropriate to use basenames for the various browser gconf
keys.
Notice the behavior of the "Preferred Applications" app. It will set
basenames, not full binary paths. Thus, xdg-settings is in a bit of conflict
with that app (the canonical method of changing preferred apps in GNOME).
Using full paths can also result in a bit of oddity when, say, using Chromium
and it calls xdg-settings after having modified its own PATH to point at
/usr/lib/chromium-browser first. It ends up setting paths that avoid its
wrapper script at /usr/bin.
I'm attaching a patch that fixes this for just GNOME. I'm not sure if other
DEs would appreciate the change, so I'm making it narrow. (The patch was
originally written for chromium's built-in version, so the diff src path
probably isn't right, but the patch should apply.)
--
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Portland-bugs mailing list
Portland-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/portland-bugs
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic