Define 'self.force' in the constructor and remove the hack in '__getattr__()'

to account for it not being defined in the constructor.

1 files changed, 6 insertions, 5 deletions

diff --git a/Lib/distutils/cmd.py b/Lib/distutils/cmd.py
index 16008c4..b3d1664 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