<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
+    if test "x$enable_glx" = xno; then<br>
+        AC_MSG_ERROR([cannot build libglvnd without GLX])<br>
+    fi<br>
+<br>
+    if test "x$enable_xlib_glx" = xyes; then<br>
+        AC_MSG_ERROR([cannot build libgvnd when Xlib-GLX is enabled])<br>
+    fi<br></blockquote><div><br></div></div>This doesn't consider the gallium-xlib implementation.  GLX configure options are specified differently now as of:<br><br>a92910a glx: Refactor the configure options for glx implementation choice (v3)<br><br></div><div class="gmail_extra">Basically all implementations are controlled via a single --enable-glx=implementation option.  If you move the logic to after the enable_glx option is validated, the you are guaranteed it's value will be one of no, dri, xlib, or gallium-xlib.  Your logic can then be:<br><span style="font-family:monospace,monospace"><br></span></div><div class="gmail_extra"><span style="font-family:monospace,monospace">case "x$enable_glx" in<br></span></div><div class="gmail_extra"><span style="font-family:monospace,monospace">xno)<br></span></div><div class="gmail_extra"><span style="font-family:monospace,monospace">  AC_MSG_ERROR([cannot build libglvnd without GLX])<br>  ;;<br>xxlib | xgallium-xlib )<br>  AC_MSG_ERROR([cannot build libgvnd when Xlib-GLX or Gallium-Xlib-GLX is enabled])<br>  ;;<br></span></div><div class="gmail_extra"><span style="font-family:monospace,monospace">xdri)<br>  ;;<br></span></div><div class="gmail_extra"><span style="font-family:monospace,monospace">esac</span><br></div><div class="gmail_extra"><br></div></div>