Re: Py_hash_t on python27

On 22.01.2016 17:40, Richard PALO wrote:
> Le 22/01/16 17:35, Rene Engelhard a écrit :
> > On Fri, Jan 22, 2016 at 05:26:05PM +0100, Richard PALO wrote:
> > > > > > -#if PY_VERSION_HEX < 0x02070000
> > > > > > +#if PY_VERSION_HEX < 0x03020000
> >
> > Whose are not supported. As said, PyUNO needs python >= 3.3 for python 3.
> > See http://cgit.freedesktop.org/libreoffice/core/tree/configure.ac#n8071
> >
> > No idea whether python 2.7 is supposed to be still supported, don't think
> > with 5.1s improved pyUNO.. I stopped building the extra pyUNO package for
> > python 2 longish ago.

yes, 2.7 is still supported, RHEL 7 doesn't have anything newer sadly.
i also remember there were some patches to build with 2.6 too from SuSE
a couple weeks ago.

> We patched that long ago, and things seem to work okay (is there something in particular to watch 
> out for?)
> because LO always seemed to pick up the wrong python...!

the configure intentionally checks only for python 3 by default because
you only want to use python 2 if you have no alternative, and you're
supposed to set environment variables like here:

http://pkgs.fedoraproject.org/cgit/rpms/libreoffice.git/tree/libreoffice.spec#n1009

> > --- configure.ac.orig        2016-01-13 01:56:20.000000000 +0000
> > +++ configure.ac
> > @@ -7997,7 +7997,7 @@ no|disable)
> > @@ -8050,9 +8050,9 @@ if test $enable_python = system; then
> >      elif test "$cross_compiling" != yes; then
> >          # Unset variables set by the above AM_PATH_PYTHON so that
> >          # we actually do check anew.
> > -        unset PYTHON am_cv_pathless_PYTHON ac_cv_path_PYTHON am_cv_python_version 
> > am_cv_python_platform am_cv_python_pythondir am_cv_python_pyexecdir
> > +        #unset PYTHON am_cv_pathless_PYTHON ac_cv_path_PYTHON am_cv_python_version 
> > am_cv_python_platform am_cv_python_pythondir am_cv_python_pyexecdir
> >          # This causes an error if no python command is found
> > -        AM_PATH_PYTHON([3.3])
> > +        AM_PATH_PYTHON([2.6])

... of course this should work too :)