Change in vdsm[master]: m4/ax_python_module.m4: do not overwrite PYTHON

danken at redhat.com danken at redhat.com
Tue May 10 11:38:36 UTC 2016


Dan Kenigsberg has uploaded a new change for review.

Change subject: m4/ax_python_module.m4: do not overwrite PYTHON
......................................................................

m4/ax_python_module.m4: do not overwrite PYTHON

the PYTHON environment variable is overly useful in the configure
script. AX_PYTHON_MODULE should not overwrite it while testing a
specific module existence.

Change-Id: I4224302b47a5f4465976af439079efd13987defa
Signed-off-by: Dan Kenigsberg <danken at redhat.com>
---
M m4/ax_python_module.m4
1 file changed, 7 insertions(+), 9 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/01/57301/1

diff --git a/m4/ax_python_module.m4 b/m4/ax_python_module.m4
index f182c48..9c78d8b 100644
--- a/m4/ax_python_module.m4
+++ b/m4/ax_python_module.m4
@@ -27,18 +27,16 @@
 
 AU_ALIAS([AC_PYTHON_MODULE], [AX_PYTHON_MODULE])
 AC_DEFUN([AX_PYTHON_MODULE],[
-    if test -z $PYTHON;
+    if test -z "$3";
     then
-        if test -z "$3";
-        then
-            PYTHON="python3"
-        else
-            PYTHON="$3"
-        fi
+        THIS_MODULE_PYTHON="python3"
+    else
+        THIS_MODULE_PYTHON="$3"
     fi
-    PYTHON_NAME=`basename $PYTHON`
+
+    PYTHON_NAME=`basename $THIS_MODULE_PYTHON`
     AC_MSG_CHECKING($PYTHON_NAME module: $1)
-    $PYTHON -c "import $1" 2>/dev/null
+    "$THIS_MODULE_PYTHON" -c "import $1" 2>/dev/null
     if test $? -eq 0;
     then
         AC_MSG_RESULT(yes)


-- 
To view, visit https://gerrit.ovirt.org/57301
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4224302b47a5f4465976af439079efd13987defa
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken at redhat.com>


More information about the vdsm-patches mailing list