[udisks] CD-ROM polling failed due to O_EXCL flag (poller.c)

PCMan pcman.tw at gmail.com
Sat Feb 26 00:53:16 PST 2011

Thanks for the suggestions.
I made a new patch reading from /proc/self/mountinfo as suggested by you.
In addition, no code is taken from HAL so there is no license issue now.
Please review the patch attached to this mail.

On Sat, Feb 26, 2011 at 12:12 AM, Kay Sievers <kay.sievers at vrfy.org> wrote:
> On Fri, Feb 25, 2011 at 14:47, PCMan <pcman.tw at gmail.com> wrote:
>> I made a small patch against the latest udisks in git after hacking HAL daemon
>> and it works. Please review the patch. Thanks a lot.
>> https://bugs.freedesktop.org/attachment.cgi?id=43798&action=edit
> mtab is officially dead now, even in mount(8). Please always use
> /proc/self/mountinfo, it has the major/minor in there, which is a far
> more reliable match than the device file name. Example is here:
>  http://git.kernel.org/?p=linux/hotplug/udev.git;a=blob;f=extras/cdrom_id/cdrom_id.c;h=4a58a49030231f854be8ce63e6b8d700b5f1b832;hb=HEAD#l118
> HAL was also GPL just like udisks, there is need to get into trouble
> with other licences added to the udisks code, especially not for 3
> lines getmntent() parsing. :)
> Tanks,
> Kay
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Fix-34710-udisks-CD-ROM-polling-failed-due-to-O_EXCL.patch
Type: text/x-patch
Size: 2350 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/devkit-devel/attachments/20110226/cb7236c9/attachment.bin>

More information about the devkit-devel mailing list