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 545025 - gvfs doesnt use hal storage.icon.drive or storage.icon.volume properties
gvfs doesnt use hal storage.icon.drive or storage.icon.volume properties
Status: RESOLVED FIXED
Product: gvfs
Classification: Core
Component: general
0.2.x
Other All
: Normal normal
: ---
Assigned To: gvfs-maint
gvfs-maint
Depends on:
Blocks:
 
 
Reported: 2008-07-27 20:08 UTC by Karl Relton
Modified: 2009-03-02 12:03 UTC
See Also:
GNOME target: ---
GNOME version: 2.21/2.22


Attachments
Patch to correct the problem (1.43 KB, patch)
2008-11-29 17:47 UTC, Karl Relton
none Details | Review

Description Karl Relton 2008-07-27 20:08:18 UTC
Please describe the problem:
The HAL specification (storage namespace section) specifies that the user can over-ride the icons to use for drives and volumes by setting the properties on an object with info.category = storage.

This used to work fine in gnome-vfs. However, with gvfs this doesn't work.

Looking at the source code, I can see it is using info.desktop.icon to specify the icon. However, this property is not listed at all in the HAL spec.

Steps to reproduce:
1. Create an fdi that matches on a disk (e.g. a USB stick), and make it merge the properties storage.icon.drive and storage.icon.volume to specify a user-chosen icon (that needs to be installed at least in the hicolor theme).
2. Insert the disk drive
3. See whether nautilus uses your chosen icon rather than the 'normal' one


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Karl Relton 2008-11-29 17:47:26 UTC
Created attachment 123650 [details] [review]
Patch to correct the problem

This patch makes the code consistent with the HAL spec (currently version 0.5.10 at http://people.freedesktop.org/~david/hal-spec/hal-spec.html).

However, other places have info.desktop.icon (info.desktop.name) which aren't covered by the spec - so it may be that it is better to change the spec!
Comment 2 Alexander Larsson 2009-03-02 12:03:19 UTC
commited.