[Python-checkins] CVS: python/dist/src/Lib/distutils cygwinccompiler.py,1.12,1.13 sysconfig.py,1.37,1.38

[M.-A. Lemburg]

Martin von Loewis wrote:
> 
> > 1. You should patch the distutils CVS tree as well.
> >
> > 2. Your fix is not backwards compatible (distutils maintains
> >    Python 1.5.2 and up compatibility, so you'll have to
> >    switch on the sys.version here).
> 
> Don't these two comments contradict each other? I cannot see why the
> distutils copy in Python needs to switch on sys.version; it is always
> in sync with its Python copy.
> 
> If the stand-alone version of distutils still needs to maintain
> backwards compatibility with older versions, a separate patch would
> need to be developed. Patching this tree with the patch I just applied
> to the Python tree would break backwards compatibility.

Discussing this really takes more time than adding the 
if sys.version < '2.2': ... 

I don't think that it's a good idea to break the synchronisity of
the distutils tree and Python's tree.
 
> Please note that Andrew Kuchling, in
> 
> http://mail.python.org/pipermail/python-dev/2001-July/016022.html
> 
> questions the need for separate distutils releases.

Not releasing new distutils versions wouldn't be in the interest
of people still using older Python versions and extension writer
who would like to use recent distutils versions for all supported
Python versions.

-- 
Marc-Andre Lemburg
CEO eGenix.com Software GmbH
______________________________________________________________________
Consulting & Company:                           http://www.egenix.com/
Python Software:                        http://www.lemburg.com/python/