On 9 October 2014 19:01, Ilia Mirkin <[email protected]> wrote: > On Thu, Oct 9, 2014 at 1:54 PM, Christian König <[email protected]> > wrote: >> From: Christian König <[email protected]> >> >> Signed-off-by: Christian König <[email protected]> >> --- >> configure.ac | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/configure.ac b/configure.ac >> index bef5753..59b90e6 100644 >> --- a/configure.ac >> +++ b/configure.ac >> @@ -1806,9 +1806,9 @@ dnl Directory for VA libs >> >> AC_ARG_WITH([va-libdir], >> [AS_HELP_STRING([--with-va-libdir=DIR], >> - [directory for the VA libraries @<:@default=`pkg-config libva >> --variable=driverdir`@:>@])], >> + [directory for the VA libraries @<:@${libdir}/dri@:>@])], >> [VA_LIB_INSTALL_DIR="$withval"], >> - [VA_LIB_INSTALL_DIR=`pkg-config libva --variable=driverdir`]) >> + [VA_LIB_INSTALL_DIR="${libdir}/dri"]) > > Seems like ${libdir}/va/drivers is more in line with the expected > libva layout, but I'm fine with it either way. > Actually the "drivers" seems to be a bit too much, but it seems that we do not really honour VA_LIB_INSTALL_DIR :\ Either way - let's get this in as is, and clean any loose ends later.
For the series Reviewed-by: Emil Velikov <[email protected]> -Emil > Thanks a lot for putting this together! Series is Reviewed-by: Ilia > Mirkin <[email protected]> _______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
