Py_hash_t on python27

[Michael Stahl]

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 :)