[PATCH libinput 2/3] Add a rules file to assign LIBINPUT_DEVICE_GROUP

[sardemff7+wayland at sardemff7.net]

On 2015-02-10 07:50, Peter Hutterer wrote:
> diff --git a/configure.ac b/configure.ac
> index f47c5a4..2420f86 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -69,6 +69,15 @@ fi
>   AC_SUBST(GCC_CFLAGS)
>   AC_SUBST(GCC_CXXFLAGS)
>
> +AC_ARG_WITH(udev-rules-dir,
> +            AS_HELP_STRING([--with-udev-rules-dir=DIR],
> +                           [Directory where udev expects its rules files
> +                           [[default=$libdir/udev/rules.d]]]),
> +            [udevdir="$withval"],

You may (should?) use $with_udev_rules_dir here. Furthermore, you should 
handle the --without case with something like that:

udev_rules_dir_default=$libdir/udev/rules.d
AC_ARG_WITH(udev-rules-dir,
             AS_HELP_STRING([--with-udev-rules-dir=DIR],
                            [Directory where udev expects its rules files
                            [[default=$udev_rules_dir_default]]]),
             [],
             [with_udev_rules_dir=yes])
case "$with_udev_rules_dir" in
     no|"") AC_MSG_ERROR([You must define an udev rules directory]) ;;
     yes) udevdir=$udev_rules_dir_default ;;
     *) udevdir=$with_udev_rules_dir ;;
esac


> index 0000000..5b39ac7
> --- /dev/null
> +++ b/rules/Makefile.am
> @@ -0,0 +1,4 @@
> +udevdir=$(UDEV_RULES_DIR)
> +udev_DATA = 80-libinput-tablet.rules
> +
> +EXTRA_DIST = $(udev_DATA)
>

Please use that only:

udevdir=$(UDEV_RULES_DIR)
dist_udev_DATA = 80-libinput-tablet.rules

-- 

Quentin “Sardem FF7” Glidic