From d197a3a12c851b59d48df1eda22cad0a570a6bd3 Mon Sep 17 00:00:00 2001 From: Greg Ward Date: Mon, 10 Apr 2000 13:11:51 +0000 Subject: [PATCH] Define 'self.force' in the constructor and remove the hack in '__getattr__()' to account for it not being defined in the constructor. --- Lib/distutils/cmd.py | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/Lib/distutils/cmd.py b/Lib/distutils/cmd.py index 16008c4fd03..b3d16648d0b 100644 --- a/Lib/distutils/cmd.py +++ b/Lib/distutils/cmd.py @@ -59,6 +59,12 @@ class Command: self._verbose = None self._dry_run = None + # Some commands define a 'self.force' option to ignore file + # timestamps, but methods defined *here* assume that + # 'self.force' exists for all commands. So define it here + # just to be safe. + self.force = None + # The 'help' flag is just used for command-line parsing, so # none of that complicated bureaucracy is needed. self.help = 0 @@ -79,11 +85,6 @@ class Command: return getattr (self.distribution, attr) else: return myval - - # Needed because some Command methods assume 'self.force' exists, - # but not all commands define 'self.force'. Ugh. - elif attr == 'force': - return None else: raise AttributeError, attr