On 08/26/2011 08:07 PM, Julien Rioux wrote:
> On 27/08/2011 1:46 AM, Enrico Forestieri wrote:
>> On Sat, Aug 27, 2011 at 01:29:37AM +0200, jri...@lyx.org wrote:
>>
>>> Author: jrioux
>>> Date: Sat Aug 27 01:29:37 2011
>>> New Revision: 39533
>>> URL: http://www.lyx.org/trac/changeset/39533
>>>
>>> Log:
>>> configure.py : subprocess, second try (check_output requires
>>> python 2.7 and throws unwanted exceptions when the process
>>> return value is not 0
>>
>> subprocess.Popen() also increases the requirement on python.
>> In README we have the following:
>>
>>      Python 2.3 or later to convert old LyX files and for helper
>> scripts.
>>      Note 1: Python 2.3.4 or later is required for exporting LyX files
>>      to a pre-unicode format, as previous versions are affected by a bug
>>      about the normalization of unicode strings.
>>      Note 2: Python 2.4.0 or later is required for the clean_dvi.py
>> script,
>>      which, however, is only used on Windows, where at least Python
>> 2.6.0
>>      is anyway recommended.
>>      Note 3: Python 3.0 or later is not supported.
>>
>> so, you may want to also update that, and next time maybe discuss such
>> changes before committing...
>>
>>
>
> You're quite right.
>
> Due to my playing with git, the commits r39525 and 39526 happened
> simultaneously, while I only meant to commit the first.
>
> subprocess.Popen itself requires 2.4, only the windows script already
> requires this python version. Is it time to bump the requirement for
> everyone?
>
I would think 2.4 was a pretty reasonable minimum, though I guess we
should check what Python is available for the more obscure OSes we claim
to support.

rh


Reply via email to