<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Apr 15, 2016 at 2:07 PM, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">On 15 April 2016 at 16:36, Chuck Atkins <<a href="mailto:chuck.atkins@kitware.com">chuck.atkins@kitware.com</a>> wrote:<br>
> Instead of cascading support for various different implementations of<br>
> GLX, all three options are now specified through the --enable-glx<br>
> option:<br>
><br>
</span>Bth having the single switch looks a lot cleaner imho. Wondering if<br>
one shouldn't use --with-glx as this is the strange we handle non<br>
"yes/no" kind of things in mesa.<br>
<br>
I'm leaning that's a bad idea since people already use --enable-glx<br></blockquote><div><br></div><div>I thought about that too but if I used --with, the it would make the most sense to split the options with --enable-glx and --wth-glx-implementation=[dri|xlib|gallium-xlib].  Since you can accomplish all of this with a single --enable option I chose to stick with reduced option complexity.<br> <br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<span class="">
>   --enable-glx[=yes]        : Defaults to dri if DRI is enabled, else<br>
>                               gallium-xlib if gallium is enabled, else<br>
>                               xlib<br>
</span>This final option changes things in a subtle way although I doubt it<br>
will cause any issues in reality.<br></blockquote><div><br></div><div>It does change it a bit, but I wanted to let somebody use ./configure --enable-glx [other options] and produce a functioning libGL.<br></div><div><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<span class="">> built causing differnt versioned and conflicting libGL libraries to be<br>
</span>"differently versioned",</blockquote><div><br></div><div>Fixed in new patch.</div><div> </div><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> installed.<br>
<br>
Please add the following to the commit message (and any r-b, acked-by... tags)<br>
Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=94086" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=94086</a></blockquote><div><br></div><div>Fixed in new patch.<br> <br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">>  AC_ARG_ENABLE([glx],<br>
> -    [AS_HELP_STRING([--enable-glx],<br>
> -        [enable GLX library @<:@default=enabled@:>@])],<br>
> +    [AS_HELP_STRING([--enable-glx[=dri|xlib|gallium-xlib]],<br>
> +        [enable the GLX library and choose an implementation<br>
> @<:@default=dri@:>@])],<br>
>      [enable_glx="$enableval"],<br>
>      [enable_glx=yes])<br>
</span>"enable_glx=dri" to reflect the help string.<br></blockquote><div><br></div><div>Changed help string to [default=auto] to better reflect that some heuristic is used to try to chose which one gets used.<br><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<span class="">
> +    xxlib | xgallium-xlib )<br>
> +        if test "x$enable_dri" = xyes; then<br>
> +            AC_MSG_ERROR([Xlib-based GLX cannot be built with DRI enabled])<br>
> +        fi<br>
</span>As the user can explicitly request gallium-xlib, in which case we<br>
should check for test -n "$with_gallium_drivers".<br></blockquote><div><br>Fixed in new patch.<br> <br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<span class="">> +*)<br>
> +    AC_MSG_ERROR([Illegal value for --enable-dri: $enable_dri])<br>
</span>Typos s/dri/glx/<br></blockquote><div><br>Fixed in new patch.<br> <br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<span class="">
> +AM_CONDITIONAL(NEED_LIBMESA, test "x$enable_glx" = xxlib -o \<br>
> +                                  "x$enable_glx" = xgallium-xlib -o \<br>
</span>I think that we don't need libmesa for the gallium one. Did you try<br>
building without it ?<br></blockquote><div><br></div><div>It seems you are correct.  Fixed in new patch.<br><br></div></div></div></div>