[PATCH] use udevadm to get device map

Kay Sievers kay.sievers at vrfy.org
Wed Aug 27 08:34:33 PDT 2008


On Wed, Aug 27, 2008 at 16:55, Dan Nicholson <dbn.lists at gmail.com> wrote:
> On Tue, Aug 26, 2008 at 8:18 PM, Lucas Hazel <lucas at die.net.au> wrote:
>> ---
>>  hald/linux/blockdev.c |    2 +-
>>  hald/linux/coldplug.c |    4 ++--
>>  2 files changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/hald/linux/blockdev.c b/hald/linux/blockdev.c
>> index 396dd9e..235e9d1 100644
>> --- a/hald/linux/blockdev.c
>> +++ b/hald/linux/blockdev.c
>> @@ -1770,7 +1770,7 @@ udev_get_device_file_for_sysfs_path (const char *sysfs_path)
>>         char *ret;
>>         char *u_stdout;
>>         int u_exit_status;
>> -        const char *argv[] = {"/usr/bin/udevinfo", "--root", "--query", "name", "--path", NULL, NULL};
>> +        const char *argv[] = {"/sbin/udevadm", "--root", "--query", "name", "--path", NULL, NULL};
>
> Don't you have to actually have to pass the "info" command to udevadm?

Yeah, it is always good to test patches. :)

And please update NEWS, to mention that we require version 117 now.
Care to send a new patch?

Thanks,
Kay


More information about the hal mailing list