[Mesa-dev] [PATCH] configure: correctly autodetect xvmc/vdpau/omx

[Emil Velikov]

On 12/06/14 07:30, Christian König wrote:
> Am 11.06.2014 23:46, schrieb Emil Velikov:
>> Commit e62b7d38a1d (configure: autodetect video state-trackers
>> when non swrast driver is present) added a check that caused
>> the autodetection to be omitted when we have the swrast gallium
>> driver. Whereas it should have skipped the VL targets when only
>> swrast was selected.
>>
>> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=79907
>> Cc: "10.2" <mesa-stable at lists.freedesktop.org>
>> Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
> 
> I was already wondering what's going wrong there. Thanks for all the work
> maintaining the build system.
> 
> Patch is Reviewed-by: Christian König <christian.koenig at amd.com>
> 
I normally have explicit configure options on my build :)

Thanks for the review, confirmed that it fixes the issue and pushed to master.
Emil

>> ---
>>   configure.ac | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/configure.ac b/configure.ac
>> index 7b0009d..a19f777 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -1412,7 +1412,7 @@ AM_CONDITIONAL(HAVE_OPENVG, test "x$enable_openvg" =
>> xyes)
>>   dnl
>>   dnl Gallium G3DVL configuration
>>   dnl
>> -if test -n "$with_gallium_drivers" && ! echo "$with_gallium_drivers" | grep
>> -q 'swrast'; then
>> +if test -n "$with_gallium_drivers" -a "x$with_gallium_drivers" != xswrast;
>> then
>>       if test "x$enable_xvmc" = xauto; then
>>       PKG_CHECK_EXISTS([xvmc], [enable_xvmc=yes], [enable_xvmc=no])
>>       fi
>