Attachment 30116 Details for Bug 24351 – Patch

[patch] Patch

0002-Fix-subsystem-check-for-firewire.patch (text/plain), 1.26 KB, created by Martin Pitt on 2009-10-06 11:59:01 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Pitt

Created: 2009-10-06 11:59:01 UTC

Size: 1.26 KB

patch

obsolete

>From d830373c5120c891983ed31dbf8031e87e209eca Mon Sep 17 00:00:00 2001
>From: Martin Pitt <martin.pitt@ubuntu.com>
>Date: Tue, 6 Oct 2009 20:54:27 +0200
>Subject: [PATCH 2/2] Fix subsystem check for firewire
>
>Firewire subsystem is called "ieee1394" in current Linux. Now check for both
>"ieee1394" and "firewire". This fixes firewire drives to not be considered
>system internal any more.
>
>https://bugs.freedesktop.org/show_bug.cgi?id=24351
>---
> src/devkit-disks-device.c |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
>diff --git a/src/devkit-disks-device.c b/src/devkit-disks-device.c
>index 6460afd..4a621c6 100644
>--- a/src/devkit-disks-device.c
>+++ b/src/devkit-disks-device.c
>@@ -1810,7 +1810,7 @@ update_drive_properties_from_sysfs (DevkitDisksDevice *device)
>                                         break;
> 
>                                 }
>-                        } else if (strcmp (subsystem, "firewire") == 0) {
>+                        } else if (strcmp (subsystem, "firewire") == 0 || strcmp (subsystem, "ieee1394") == 0) {
> 
>                                 /* TODO: krh has promised a speed file in sysfs; theoretically, the speed can
>                                  *       be anything from 100, 200, 400, 800 and 3200. Till then we just hardcode
>-- 
>1.6.3.3
>

Actions: View | Splinter Review

Attachments on bug 24351: 30116

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.