Explicitly check for python3 in configure.ac

[libvirt] [PATCH sandbox] Explicitly check for python3 in configure.ac

Posted by Daniel P. Berrangé 6 years ago

A bare AM_PATH_PYTHON statement is satisfied by any version of python >=
2.0.0, but we converted to Python 3, so should be explicit about it.

Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---
 configure.ac | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/configure.ac b/configure.ac
index cb7a483..28305d2 100644
--- a/configure.ac
+++ b/configure.ac
@@ -130,7 +130,7 @@ dnl Should be in m4/virt-gettext.m4 but intltoolize is too
 dnl dumb to find it there
 IT_PROG_INTLTOOL([0.35.0])
 
-AM_PATH_PYTHON
+AM_PATH_PYTHON([3])
 
 AC_OUTPUT(Makefile
           libvirt-sandbox/Makefile
-- 
2.14.3

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH sandbox] Explicitly check for python3 in configure.ac

Posted by Cedric Bosdonnat 6 years ago

On Fri, 2018-03-23 at 15:57 +0000, Daniel P. Berrangé wrote:
> A bare AM_PATH_PYTHON statement is satisfied by any version of python >=
> 2.0.0, but we converted to Python 3, so should be explicit about it.
> 
> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
> ---
>  configure.ac | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/configure.ac b/configure.ac
> index cb7a483..28305d2 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -130,7 +130,7 @@ dnl Should be in m4/virt-gettext.m4 but intltoolize is too
>  dnl dumb to find it there
>  IT_PROG_INTLTOOL([0.35.0])
>  
> -AM_PATH_PYTHON
> +AM_PATH_PYTHON([3])
>  
>  AC_OUTPUT(Makefile
>            libvirt-sandbox/Makefile

ACK
--
Cedric

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list