[Mesa-dev] [PATCH 1/2] configure: enable glx-tls by default

[Andreas Boll]

2016-12-05 20:02 GMT+01:00 Emil Velikov <emil.l.velikov at gmail.com>:
> From: Emil Velikov <emil.velikov at collabora.com>
>
> In the (not too) distant future we'd want to remove this option and
> effectively drop the other codepath(s) we have in our dispatch.
>
> Linux distributions have been using --enable-glx-tls for a number of
> years. Some/most BSD platforms still don't support this, yet this should
> serve as an encouragement to move things forwards.
>
> Cc: Jean-Sébastien Pédron <dumbbell at FreeBSD.org>
> Cc: Jonathan Gray <jsg at jsg.id.au>
> Cc: mesa-maintainers at lists.freedesktop.org
> Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
> ---
> Gents, please forward to the respective lists and fellow maintainers. Be
> that on your or brother/sister distro.
>
> Joining up on mesa-maintainers will be appreciated.
> ---
>  configure.ac | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/configure.ac b/configure.ac
> index adca49d..52bd9a6 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1707,7 +1707,7 @@ AC_ARG_ENABLE([glx-tls],
>      [AS_HELP_STRING([--enable-glx-tls],
>          [enable TLS support in GLX @<:@default=disabled@:>@])],

Please update the help string to reflect the new default behavior.

With that fixed:

Reviewed-by: Andreas Boll <andreas.boll.dev at gmail.com>

>      [GLX_USE_TLS="$enableval"],
> -    [GLX_USE_TLS=no])
> +    [GLX_USE_TLS=yes])
>  AC_SUBST(GLX_TLS, ${GLX_USE_TLS})
>
>  AS_IF([test "x$GLX_USE_TLS" = xyes -a "x$ax_pthread_ok" = xyes],
> --
> 2.10.2
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev