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