[RFC PATCH] Initial libudev input-hotplug support
Julien Cristau
jcristau at debian.org
Wed Sep 30 01:17:47 PDT 2009
On Wed, Sep 30, 2009 at 08:47:27 +0200, Rémi Cardona wrote:
> Hi Julien,
>
> Le mardi 29 septembre 2009 à 23:54 +0200, Julien Cristau a écrit :
> > If libudev is found, we use that for hotplug and disable the hal and
> > dbus backends.
>
> Why disable dbus? I see udev being in conflict with HAL, but not with
> the dbus code.
Yeah, I could do that.
> Not that it matters much, I don't think any tool/utility
> uses the dbus interfaces anyway. Maybe it's time to ditch it?
>
[...]
> > +
> > +#define UDEV_PROP_KEY "x11_options."
> > +#define UDEV_XKB_PROP_KEY "xkb."
>
> <harmless bikeshedding>
> Why not merge the 2 into a single :
>
> UDEV_PROP_KEY "xorg."
> </bikeshedding>
>
> I think the "xkb" HAL keys were deprecated in favor of
> "x11_options.Xkb*", let's at least drop the deprecated one.
>
Well my understanding was that input.xkb.foo was preferred. There's
nothing strictly x11-specific to the xkb keys (we use the
xkeyboard-config data files to set the console keymap, e.g.).
[...]
> > +AC_ARG_ENABLE(config-udev, AS_HELP_STRING([--enable-config-udev], [Build udev support (default: auto)]), [CONFIG_UDEV=$enableval], [CONFIG_UDEV=auto])
> > AC_ARG_ENABLE(config-dbus, AS_HELP_STRING([--enable-config-dbus], [Build D-BUS API support (default: no)]), [CONFIG_DBUS_API=$enableval], [CONFIG_DBUS_API=no])
> > AC_ARG_ENABLE(config-hal, AS_HELP_STRING([--disable-config-hal], [Build HAL support (default: auto)]), [CONFIG_HAL=$enableval], [CONFIG_HAL=auto])
> > AC_ARG_ENABLE(xfree86-utils, AS_HELP_STRING([--enable-xfree86-utils], [Build xfree86 DDX utilities (default: enabled)]), [XF86UTILS=$enableval], [XF86UTILS=yes])
> > @@ -742,6 +743,25 @@ LIBXI="xi >= 1.2.99.1"
> > LIBPCIACCESS="pciaccess >= 0.8.0"
> > LIBGLIB="glib-2.0 >= 2.16"
>
> Maybe we could have --enable-config={dbus,hal,udev,none} instead?
>
> Anyhow, I don't have much else to say about the real libudev code except
> that it's well indented ;)
>
Thanks,
Julien
More information about the xorg-devel
mailing list