Check that 'self.formats' is good early on.

diff --git a/Lib/distutils/command/sdist.py b/Lib/distutils/command/sdist.py
index 2cf8e385b84c25e50fa67362a8ab868f34f28794..8aa4618eccf7daf9b851760993ea457f9dff26c3 100644 (file)
--- a/Lib/distutils/command/sdist.py
+++ b/Lib/distutils/command/sdist.py
@@ -11,10 +11,10 @@ import fnmatch
 from types import *
 from glob import glob
 from distutils.core import Command
-from distutils.util import \
-     newer, create_tree, remove_tree, make_tarball, make_zipfile, native_path
+from distutils.util import newer, create_tree, remove_tree, native_path
+from distutils.archive_util import check_archive_formats
 from distutils.text_file import TextFile
-from distutils.errors import DistutilsExecError
+from distutils.errors import DistutilsExecError, DistutilsOptionError
 
 
 class sdist (Command):
@@ -81,6 +81,11 @@ class sdist (Command):
         elif type (self.formats) is StringType:
             self.formats = string.split (self.formats, ',')
 
+        bad_format = check_archive_formats (self.formats)
+        if bad_format:
+            raise DistutilsOptionError, \
+                  "unknown archive format '%s'" % bad_format
+
 
     def run (self):