[Python-Dev] Re: [Python-checkins] CVS: distutils/distutils cmd.py
Greg Ward
gward@mems-exchange.org
Mon, 10 Apr 2000 09:11:02 -0400
On 10 April 2000, Greg Stein said:
> On Sun, 9 Apr 2000, Greg Ward wrote:
> > Modified Files:
> > cmd.py
> > Log Message:
> > Added a check for the 'force' attribute in '__getattr__()' -- better than
> > crashing when self.force not defined.
>
> This seems a bit silly. Why don't you simply define .force in the __init__
> method?
Duhh, 'cause I'm stupid? No, that's not it. 'Cause I was doing this on
a lazy Sunday evening and not really thinking about it? Yeah, I think
that's it.
There, I now define self.force in the Command class constructor. A wee
bit cheesy (not all Distutils command classes need or use self.force,
and it wouldn't always mean the same thing), but it means minimal code
upheaval for now.
> [ damn... can't see the code... went and checked it out... ]
Oops, that was a CVS config thing. Fixed now -- I'll go checkin that
change and we'll all see if it worked. Just as well it was off though
-- I checked in a couple of big documentation updates this weekend, and
who wants to see 30k of LaTeX patches in their inbox on Monday morning?
;-)
Greg
--
Greg Ward - software developer gward@mems-exchange.org
MEMS Exchange / CNRI voice: +1-703-262-5376
Reston, Virginia, USA fax: +1-703-262-5367