Summary: | UDisksDevice::queryDeviceInterface has bogus implementation for Block device. Makes k3b not show dvd writers | ||
---|---|---|---|
Product: | [Unmaintained] solid | Reporter: | cristi.posoiu |
Component: | libsolid-udisks | Assignee: | Lukáš Tinkl <lukas> |
Status: | RESOLVED FIXED | ||
Severity: | crash | CC: | maggsimo |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Mandriva RPMs | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: |
Description
cristi.posoiu
2011-01-02 01:28:00 UTC
SVN commit 1210957 by lukas: don't ignore an empty optical disc BUG: 261834 M +1 -1 udisksdevice.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1210957 *** Bug 261618 has been marked as a duplicate of this bug. *** I'm not sure I understand the fix. Was this tested - i.e. does k3b now show the drive always, *regardless* of the unit having anything put in it? To me it looks like it won't since k3b will still won't be able to get a Solid::Block interface from it. I.e. I still see that returning that interface is conditioned by the value of DeviceBlockSize udisk property - which in turn I don't think it is having anything to do with the isDeviceBlacklisted function. I consider that the Solid::Block interface should be available (since the device is always present there, /dev/sro in my case) regardless of the unit having or not something in it (blank or not blank). Unfortunately I don't have yet a way to test kde from svn so that I test the fix so all my comments are based solely on the code. *** Bug 261835 has been marked as a duplicate of this bug. *** Yes, I tested it with both blank CD and DVD in the drive, K3b correctly reported it as such. I was talking about k3b seeing the writer *without* anything in it. Oh, sry for the misunderstanding :) Gonna test again SVN commit 1211252 by lukas: fix the check for a Block device, according to the udisks docu (fixes k3b not seeing an empty writer, w/o CD or DVD) BUG: 261834 M +1 -1 udisksdevice.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1211252 This change might fix it (it is kind of the 'always return true' suggestion) Thanks! |