Yaniv Bronhaim has uploaded a new change for review.

Change subject: Enable python3 over fedora as default and disable it over 
centos\rhel
......................................................................

Enable python3 over fedora as default and disable it over centos\rhel

Until RHEL 8 which will include py3 packages as default, we prefer to
disable python3 build and test run as default.

Change-Id: I610e4c0d27f12ab37b9dd5b1f5a53f5729627054
Signed-off-by: Yaniv Bronhaim <[email protected]>
---
M Makefile.am
M configure.ac
2 files changed, 9 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/72/56072/1

diff --git a/Makefile.am b/Makefile.am
index 4ce5ec5..de7cd31 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -127,7 +127,7 @@
        fi
 
 # Note: dependencies ordered by time needed to run them
-check-recursive: gitignore abs_imports python3 pyflakes pep8
+check-recursive: gitignore abs_imports python3
 
 .PHONY: check-all
 check-all:
diff --git a/configure.ac b/configure.ac
index 57b4a32..fd799fe 100644
--- a/configure.ac
+++ b/configure.ac
@@ -70,14 +70,20 @@
       AC_SUBST([VHOSTMD], ['1']),
       AC_SUBST([VHOSTMD], ['0']))
 
+if grep -q 'Fedora' /etc/redhat-release; then
+       fedora_run="yes"
+else
+       fedora_run="no"
+fi
+
 AC_ARG_ENABLE(
     [python3],
     [AS_HELP_STRING(
         [--disable-python3],
-        [disable build for python3 @<:@default=no@:>@]
+        [disable build for python3. Disabled by default over centos and RHEL]
     )],
     ,
-    [enable_python3="yes"]
+    [enable_python3=${fedora_run}]
 )
 AS_IF([test "${enable_python3}" = "yes"],
       AC_SUBST([PYTHON3_SUPPORT], ['1']),


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I610e4c0d27f12ab37b9dd5b1f5a53f5729627054
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to