Moved warnings out of 'finalize_options()' into 'run()'.
Added a warning for 'bdist_base' directory.
This commit is contained in:
parent
9581144329
commit
0eeebfda98
|
@ -34,13 +34,6 @@ class clean (Command):
|
||||||
self.all = None
|
self.all = None
|
||||||
|
|
||||||
def finalize_options(self):
|
def finalize_options(self):
|
||||||
if self.build_lib and not os.path.exists (self.build_lib):
|
|
||||||
self.warn ("'%s' does not exist -- can't clean it" %
|
|
||||||
self.build_lib)
|
|
||||||
if self.build_temp and not os.path.exists (self.build_temp):
|
|
||||||
self.warn ("'%s' does not exist -- can't clean it" %
|
|
||||||
self.build_temp)
|
|
||||||
|
|
||||||
self.set_undefined_options('build',
|
self.set_undefined_options('build',
|
||||||
('build_base', 'build_base'),
|
('build_base', 'build_base'),
|
||||||
('build_lib', 'build_lib'),
|
('build_lib', 'build_lib'),
|
||||||
|
@ -53,11 +46,21 @@ class clean (Command):
|
||||||
# gone)
|
# gone)
|
||||||
if os.path.exists (self.build_temp):
|
if os.path.exists (self.build_temp):
|
||||||
remove_tree (self.build_temp, self.verbose, self.dry_run)
|
remove_tree (self.build_temp, self.verbose, self.dry_run)
|
||||||
|
else:
|
||||||
|
self.warn ("'%s' does not exist -- can't clean it" %
|
||||||
|
self.build_temp)
|
||||||
|
|
||||||
|
|
||||||
|
|
||||||
|
|
||||||
if self.all:
|
if self.all:
|
||||||
# remove the module build directory (unless already gone)
|
# remove the module build directory (unless already gone)
|
||||||
if os.path.exists (self.build_lib):
|
if os.path.exists (self.build_lib):
|
||||||
remove_tree (self.build_lib, self.verbose, self.dry_run)
|
remove_tree (self.build_lib, self.verbose, self.dry_run)
|
||||||
|
else:
|
||||||
|
self.warn ("'%s' does not exist -- can't clean it" %
|
||||||
|
self.build_lib)
|
||||||
|
|
||||||
# remove the temporary directory used for creating built
|
# remove the temporary directory used for creating built
|
||||||
# distributions (default "build/bdist") -- eg. type of
|
# distributions (default "build/bdist") -- eg. type of
|
||||||
# built distribution will have its own subdirectory under
|
# built distribution will have its own subdirectory under
|
||||||
|
@ -65,6 +68,9 @@ class clean (Command):
|
||||||
# 'remove_tree()'.
|
# 'remove_tree()'.
|
||||||
if os.path.exists (self.bdist_base):
|
if os.path.exists (self.bdist_base):
|
||||||
remove_tree (self.bdist_base, self.verbose, self.dry_run)
|
remove_tree (self.bdist_base, self.verbose, self.dry_run)
|
||||||
|
else:
|
||||||
|
self.warn ("'%s' does not exist -- can't clean it" %
|
||||||
|
self.bdist_base)
|
||||||
|
|
||||||
# just for the heck of it, try to remove the base build directory:
|
# just for the heck of it, try to remove the base build directory:
|
||||||
# we might have emptied it right now, but if not we don't care
|
# we might have emptied it right now, but if not we don't care
|
||||||
|
|
Loading…
Reference in New Issue