[PATCH] use udevadm to get device map

Danny Kukawka danny.kukawka at web.de
Wed Aug 27 09:25:20 PDT 2008


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)?

Danny


More information about the hal mailing list