[PATCH] use udevadm to get device map

[Kay Sievers]

On Wed, Aug 27, 2008 at 18:25, Danny Kukawka <danny.kukawka at web.de> wrote:
> On Mittwoch, 27. August 2008, Kay Sievers wrote:
>> 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?
>
> Is the dependency to 117 really needed (except the fact that udevinfo is no
> longer available in newer udev versions)?

Not sure, but it is reason enough to bump the requirement. Udev 117 is
9 months old, and depending on that with the next release of HAL is
fine.

Thanks,
Kay