[Mesa-dev] [PATCH 1/2] configure.ac: move AC_MSG_RESULT reportging back into the m4 macro
Emil Velikov
emil.l.velikov at gmail.com
Mon Mar 23 10:49:23 PDT 2015
The one who does AC_MSG_CHECKING should provide the AC_MSG_RESULT.
Fixed: ced9425327b (configure: Introduce new output variable to
ax_check_python_mako_module.m4"
Cc: Samuel Iglesias Gonsalvez <siglesias at igalia.com>
Cc: "10.5" <mesa-stable at lists.freedesktop.org>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89328
Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
---
Strictly speaking not a hard requirement for 10.5, although the original
commit which resolves the bug is missing the Cc and Bugzilla tags. This
will allow us to kill two birds with one stone.
-Emil
configure.ac | 4 +---
m4/ax_check_python_mako_module.m4 | 2 ++
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/configure.ac b/configure.ac
index 08378f5..3e5b6f5 100644
--- a/configure.ac
+++ b/configure.ac
@@ -123,9 +123,7 @@ if test "x$INDENT" != "xcat"; then
fi
AX_CHECK_PYTHON_MAKO_MODULE($PYTHON_MAKO_REQUIRED)
-if test -n "$PYTHON2" -a "x$acv_mako_found" = "xyes"; then
- AC_MSG_RESULT(yes)
-else
+if test -n "$PYTHON2" -a "x$acv_mako_found" != "xyes"; then
AC_MSG_ERROR([Python mako module v$PYTHON_MAKO_REQUIRED or higher not found])
fi
diff --git a/m4/ax_check_python_mako_module.m4 b/m4/ax_check_python_mako_module.m4
index ee68a7c..7d9bb51 100644
--- a/m4/ax_check_python_mako_module.m4
+++ b/m4/ax_check_python_mako_module.m4
@@ -54,8 +54,10 @@ else:
" | $PYTHON2 -
if test $? -ne 0 ; then
+ AC_MSG_RESULT(no)
AC_SUBST(acv_mako_found, 'no')
else
+ AC_MSG_RESULT(yes)
AC_SUBST(acv_mako_found, 'yes')
fi
])
--
2.3.1
More information about the mesa-dev
mailing list