This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author pje
Recipients barry, gvanrossum, ned.deily, pje, ssteiner, tarek
Date 2009-10-05.18:35:58
SpamBayes Score 8.881784e-16
Marked as misclassified No
Message-id <1254767760.77.0.992969237938.issue7064@psf.upfronthosting.co.za>
In-reply-to
Content
I appreciate the change being made.  The specific code in setuptools is
relying on an erroneous reading of get_ext_filename()'s docstring (which
as of 2.3 at least, implied to me at least that it is always called with
an extension's full dotted name), and so I will fix it in setuptools as
well.

I would like to point out, however, that subclassing is not
"monkeypatching".  The distutils explicitly documents the ability to
extend and subclass commands, and this particular subclassing was based
on code provided by another distutils user, who'd extended build_ext to
support building dynamic libraries.

It is likely that that user, and anyone else who extended the build_ext
command in a similar way (e.g. numpy's and Twisted's distutils
extensions), could be affected by the change.

So, this was neither a matter of monkeypatching, nor special treatment
for setuptools.  Setuptools is far from the first or last distutils
extension ever written, and build_ext tends to get extended more than
most other commands.  It's important to remember subclassing use cases
when modifying distutils commands; subclasses inherently rely on a
stricter interface than a class's public API.
History
Date User Action Args
2009-10-05 18:36:00pjesetrecipients: + pje, gvanrossum, barry, ssteiner, tarek, ned.deily
2009-10-05 18:36:00pjesetmessageid: <1254767760.77.0.992969237938.issue7064@psf.upfronthosting.co.za>
2009-10-05 18:35:59pjelinkissue7064 messages
2009-10-05 18:35:58pjecreate