[Mesa-dev] [PATCH 4/4] configure: use $libdir/dri as default for VA-API

[Emil Velikov]

On 9 October 2014 19:01, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
> On Thu, Oct 9, 2014 at 1:54 PM, Christian König <deathsimple at vodafone.de> wrote:
>> From: Christian König <christian.koenig at amd.com>
>>
>> Signed-off-by: Christian König <christian.koenig at amd.com>
>> ---
>>  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 <emil.l.velikov at gmail.com>

-Emil

> Thanks a lot for putting this together! Series is Reviewed-by: Ilia
> Mirkin <imirkin at alum.mit.edu>