GNOME Bugzilla – Bug 659109
Mark bootable mounts as such
Last modified: 2011-09-19 14:53:15 UTC
Add appropriate content-type for bootable mounts. Although 'bootability' information is coming from libosinfo[1], we are not adding a dependency on it with this patch since libosinfo sets a boolean property on the udev device for us. We can then just query that property using gudev. [1] https://fedorahosted.org/libosinfo/
Created attachment 196573 [details] [review] gdu volume monitor: Mark bootable mounts as such
It should be a udisks property instead, I would think. Looking up devices by device node doesn't sound good when you have access to its sysfs path.
(In reply to comment #2) > It should be a udisks property instead, I would think. I patched udisks first but David Zeuthen insisted that this go in gvfs and Alex was ok with that. > Looking up devices by > device node doesn't sound good when you have access to its sysfs path. I don't see any gdu_device_get_sysfs_path().
udisks shows things like: Showing information for /org/freedesktop/UDisks/devices/sr0 native-path: /sys/devices/pci0000:00/0000:00:1f.2/host1/target1:0:0/1:0:0:0/block/sr0 device: 11:0 device-file: /dev/sr0 presentation: /dev/sr0 by-id: /dev/disk/by-id/ata-Slimtype_BD_E_DL4ETS_1037763 by-path: /dev/disk/by-path/pci-0000:00:1f.2-scsi-1:0:0:0 Not certain how those are exposed in gdu.
Created attachment 196948 [details] [review] gdu volume monitor: Mark bootable mounts as such Add appropriate content-type for bootable mounts. Although 'bootability' information is coming from libosinfo[1], we are not adding a dependency on it with this patch since libosinfo sets a boolean property on the udev device for us. We can then just query that property using gudev. [1] https://fedorahosted.org/libosinfo/
pushed.