Is it me, or is it autotools being silly?

[David Sommerseth]

Hi,

So I'm running some builds for the openvpn3-linux project on Fedora 
Rawhide ...  And we have this little line in configure.ac:

     AM_PATH_PYTHON([3.5],, [:])

When ./configure runs, it reports:

     checking for python3 version... 3.1

As a colleague said, mathematically this is correct.  But that doesn't 
really help.  Who/what is truncating Python 3.10 to 3.1?  And how can 
that be fixed?  Is someone already looking into this?


Complete build logs can be seen here:
<https://download.copr.fedorainfracloud.org/results/dsommers/openvpn3/fedo...


-- 
kind regards,

David Sommerseth
OpenVPN Inc

[Miro Hrončok]

On 14. 07. 21 11:49, David Sommerseth wrote:
>  
>  Hi,
>  
>  So I'm running some builds for the openvpn3-linux project on Fedora Rawhide 
>  ...  And we have this little line in configure.ac:
>  
>       AM_PATH_PYTHON([3.5],, [:])
>  
>  When ./configure runs, it reports:
>  
>       checking for python3 version... 3.1
>  
>  As a colleague said, mathematically this is correct.  But that doesn't really 
>  help.  Who/what is truncating Python 3.10 to 3.1?  And how can that be fixed?  
>  Is someone already looking into this? 
Is the configure script regenerated during build? Old consifure scripts tend to 
do this.

This was fixed e.g. in automake:

https://bugzilla.redhat.com/show_bug.cgi?id=1889732

But depending on how is the configure script created, might require fix in the 
project itself.

I suggest to grep the configure script for 'sys.version' and see what it does 
with that value (e.g. sys.version[:3] is usually the problem).

-- 
Miro Hrončok
-- 
Phone: [phone]
IRC: mhroncok

[David Sommerseth]

On 14/07/2021 11:55, Miro Hrončok wrote:
>  On 14. 07. 21 11:49, David Sommerseth wrote:
> >
> > Hi,
> >
> > So I'm running some builds for the openvpn3-linux project on Fedora Rawhide
> > ...  And we have this little line in configure.ac:
> >
> >       AM_PATH_PYTHON([3.5],, [:])
> >
> > When ./configure runs, it reports:
> >
> >       checking for python3 version... 3.1
> >
> > As a colleague said, mathematically this is correct.  But that doesn't really
> > help.  Who/what is truncating Python 3.10 to 3.1?  And how can that be fixed?
> > Is someone already looking into this?
>  
>  Is the configure script regenerated during build? Old consifure scripts tend to
>  do this. 
No, the tarball is pre-generated - on an Ubuntu 20.04 box, iirc.

>  This was fixed e.g. in automake:
>  
>  https://bugzilla.redhat.com/show_bug.cgi?id=1889732
>  
>  But depending on how is the configure script created, might require fix in the
>  project itself. 
Thanks!  This was the pointer I needed and didn't find myself.  I'll dig 
into this.

>  I suggest to grep the configure script for 'sys.version' and
> see what it does
>  with that value (e.g. sys.version[:3] is usually the problem). 
Yupp, the configure script does contain sys.version[:3].

Thanks for quick and good feedback!  I'll probably just add a re-run of 
autoreconf before %configure in the .spec file to circumvent this for 
now until we get automake upgraded.


-- 
kind regards,

David Sommerseth

[Richard W.M. Jones]

On Wed, Jul 14, 2021 at 11:49:23AM +0200, David Sommerseth wrote:
>  
>  Hi,
>  
>  So I'm running some builds for the openvpn3-linux project on Fedora
>  Rawhide ...  And we have this little line in configure.ac:
>  
>      AM_PATH_PYTHON([3.5],, [:])
>  
>  When ./configure runs, it reports:
>  
>      checking for python3 version... 3.1
>  
>  As a colleague said, mathematically this is correct.  But that
>  doesn't really help.  Who/what is truncating Python 3.10 to 3.1?
>  And how can that be fixed?  Is someone already looking into this? 
It's a bug in AM_PATH_PYTHON (defined in
/usr/share/aclocal-1.16/python.m4).

  AC_CACHE_CHECK([for $am_display_PYTHON version], [am_cv_python_version],
    [am_cv_python_version=`$PYTHON -c "import sys;
sys.stdout.write(sys.version[[:3]])"`])

In the Python 3.10 interpreter:

> >> print(sys.version[:3]) 
3.1

(because the first 3 characters of the string "3.10.0b2" are "3.1")

This seems to have been fixed upstream already:

https://git.savannah.gnu.org/cgit/automake.git/tree/m4/python.m4#n89
https://git.savannah.gnu.org/cgit/automake.git/commit/m4/python.m4?id=e21...

Rich.

-- 
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
virt-builder quickly builds VMs from scratch
http://libguestfs.org/virt-builder.1.html

[Ondrej Dubaj]

This should be also fixed in Fedora already

https://src.fedoraproject.org/rpms/automake/c/3327906e94b767330433fda3da2...

Ondrej

On Wed, Jul 14, 2021 at 12:11 PM Richard W.M. Jones <rjones(a)redhat.com&gt;
wrote:

>  On Wed, Jul 14, 2021 at 11:49:23AM +0200, David Sommerseth wrote:
>  >
>  > Hi,
>  >
>  > So I'm running some builds for the openvpn3-linux project on Fedora
>  > Rawhide ...  And we have this little line in configure.ac:
>  >
>  >     AM_PATH_PYTHON([3.5],, [:])
>  >
>  > When ./configure runs, it reports:
>  >
>  >     checking for python3 version... 3.1
>  >
>  > As a colleague said, mathematically this is correct.  But that
>  > doesn't really help.  Who/what is truncating Python 3.10 to 3.1?
>  > And how can that be fixed?  Is someone already looking into this?
>
>  It's a bug in AM_PATH_PYTHON (defined in
>  /usr/share/aclocal-1.16/python.m4).
>
>    AC_CACHE_CHECK([for $am_display_PYTHON version], [am_cv_python_version],
>      [am_cv_python_version=`$PYTHON -c "import sys;
>  sys.stdout.write(sys.version[[:3]])"`])
>
>  In the Python 3.10 interpreter:
>
>  >>> print(sys.version[:3])
>  3.1
>
>  (because the first 3 characters of the string "3.10.0b2" are "3.1")
>
>  This seems to have been fixed upstream already:
>
>  https://git.savannah.gnu.org/cgit/automake.git/tree/m4/python.m4#n89
>
>
> https://git.savannah.gnu.org/cgit/automake.git/commit/m4/python.m4?id=e21...
>
>  Rich.
>
>  --
>  Richard Jones, Virtualization Group, Red Hat
>  http://people.redhat.com/~rjones
>  Read my programming and virtualization blog: http://rwmj.wordpress.com
>  virt-builder quickly builds VMs from scratch
>  http://libguestfs.org/virt-builder.1.html
>  _______________________________________________
>  devel mailing list -- devel(a)lists.fedoraproject.org
>  To unsubscribe send an email to devel-leave(a)lists.fedoraproject.org
>  Fedora Code of Conduct:
>  https://docs.fedoraproject.org/en-US/project/code-of-conduct/
>  List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
>  List Archives:
>  https://lists.fedoraproject.org/archives/list/[e-mail]
>  Do not reply to spam on the list, report it:
>  https://pagure.io/fedora-infrastructure
>