On 19 January 2017 at 00:18, Steven Newbury <st...@snewbury.org.uk> wrote:
> On Wed, 2016-12-07 at 13:25 +0000, Emil Velikov wrote:
>> From: Emil Velikov <emil.veli...@collabora.com>
>>
>> We've been using and depending on it for at least a couple of years.
>> Make it obvious and error out, should one opt for it.
>>
>> Signed-off-by: Emil Velikov <emil.veli...@collabora.com>
>> ---
>>  configure.ac | 4 +---
>>  1 file changed, 1 insertion(+), 3 deletions(-)
>>
>> diff --git a/configure.ac b/configure.ac
>> index 162c31a..f1f41db 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -1577,9 +1577,7 @@ if test "x$enable_shader_cache" = "xyes"; then
>>  fi
>>
>>  if test "x$enable_dri" = xyes; then
>> -    if test "$enable_static" = yes; then
>> -        AC_MSG_ERROR([Cannot use static libraries for DRI drivers])
>> -    fi
>> +    require_dri_shared_libs_and_glapi "DRI"
>>
>>      # not a hard requirement as swrast does not depend on it
>>      if test "x$have_libdrm" = xyes; then
>
> What is this change supposed to do?  It just gives the below error
> here:
>
Add an extra "Thou Shalt Not use static glapi" on top of the existing
cannot build with static.

> require_dri_shared_libs_and_glapi: command not found
Should have moved the function further up. Patch coming in a second.

Thanks
Emil
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to