Mesa (master): configure: avoid constantly building megadrivers 'core'

Emil Velikov evelikov at kemper.freedesktop.org
Fri Feb 28 22:56:50 UTC 2014


Module: Mesa
Branch: master
Commit: f42333b6b6f17d4f77bcc6dfc5dd9c9e38e8c99e
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=f42333b6b6f17d4f77bcc6dfc5dd9c9e38e8c99e

Author: Emil Velikov <emil.l.velikov at gmail.com>
Date:   Mon Feb 24 22:57:59 2014 +0000

configure: avoid constantly building megadrivers 'core'

The issue is caused by a thinko that an empty string will be
considered of zero length by 'test'. This is not the case,
thus we were building the 'core' of megadrivers even when no
classic drivers were built.

Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>

---

 configure.ac |   11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/configure.ac b/configure.ac
index 0e0fd18..1688aae 100644
--- a/configure.ac
+++ b/configure.ac
@@ -965,6 +965,9 @@ if test "x$with_dri_drivers" = xauto; then
         with_dri_drivers="no"
     fi
 fi
+if test "x$with_dri_drivers" = xno; then
+    with_dri_drivers=''
+fi
 
 dnl If $with_dri_drivers is yes, drivers will be added through
 dnl platform checks. Set DEFINES and LIB_DEPS
@@ -1043,9 +1046,9 @@ fi
 AC_SUBST([DRI_LIB_DEPS])
 AC_SUBST([GALLIUM_DRI_LIB_DEPS])
 
-DRI_DIRS=""
-dnl Duplicates in DRI_DIRS are removed by sorting it after this block
-if test "x$with_dri_drivers" != xno; then
+DRI_DIRS=''
+dnl Duplicates in DRI_DIRS are removed by sorting it at the end of this block
+if test -n "$with_dri_drivers"; then
     if test "x$enable_opengl" != xyes; then
         AC_MSG_ERROR([--with-dri-drivers requires OpenGL])
     fi
@@ -1082,8 +1085,8 @@ if test "x$with_dri_drivers" != xno; then
             ;;
         esac
     done
+    DRI_DIRS=`echo $DRI_DIRS|tr " " "\n"|sort -u|tr "\n" " "`
 fi
-DRI_DIRS=`echo $DRI_DIRS|tr " " "\n"|sort -u|tr "\n" " "`
 
 AM_CONDITIONAL(NEED_MEGADRIVER, test -n "$DRI_DIRS")
 AM_CONDITIONAL(NEED_LIBMESA, test "x$enable_xlib_glx" = xyes -o \




More information about the mesa-commit mailing list