rhuij...@apache.org wrote on Sat, Jul 20, 2013 at 10:44:32 -0000: > Author: rhuijben > Date: Sat Jul 20 10:44:31 2013 > New Revision: 1505121 > > URL: http://svn.apache.org/r1505121 > Log: > * build/generator/gen_base.py > (write_errno_table): Remove obsolete part of comment. On Windows we always > required python and on other platforms we require python for generating > the > sql headers since 1.7. >
The unix part wasn't and isn't obsolete. We require Python at autogen time, but the comment said (correctly) that Python isn't required at configure time. > Modified: > subversion/trunk/build/generator/gen_base.py > > Modified: subversion/trunk/build/generator/gen_base.py > URL: > http://svn.apache.org/viewvc/subversion/trunk/build/generator/gen_base.py?rev=1505121&r1=1505120&r2=1505121&view=diff > ============================================================================== > --- subversion/trunk/build/generator/gen_base.py (original) > +++ subversion/trunk/build/generator/gen_base.py Sat Jul 20 10:44:31 2013 > @@ -244,9 +244,7 @@ class GeneratorBase: > # ### > # ### Currently it's only used by maintainer-mode builds. If this > # ### functionality ever moves to release builds, it will have to move > - # ### to configure-time. (But then you have to solve two problems: > - # ### what to do on windows, and what to do on unix when Python is not > - # ### available at configure-time.) > + # ### to configure-time. > import errno > fd = open('subversion/libsvn_subr/errorcode.inc', 'w') > fd.write('/* This file was generated by build/generator/gen_base.py > */\n\n') > >