From: Tarek Ziadé <ziade.tarek@gmail.com>
Date: Fri, 17 Apr 2009 14:34:49 +0000 (+0000)
Subject: Merged revisions 71674 via svnmerge from
X-Git-Tag: v3.1b1~264
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=f633ff4546a4d044803ea83901e8c4ea0ee60e87;p=python

Merged revisions 71674 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/trunk

........
  r71674 | tarek.ziade | 2009-04-17 16:29:56 +0200 (Fri, 17 Apr 2009) | 1 line

  DistutilsSetupError was not raised when one single warning occured
........
---

diff --git a/Lib/distutils/command/check.py b/Lib/distutils/command/check.py
index 5dd73b07af..9a8fca1d5f 100644
--- a/Lib/distutils/command/check.py
+++ b/Lib/distutils/command/check.py
@@ -73,7 +73,7 @@ class check(Command):
 
         # let's raise an error in strict mode, if we have at least
         # one warning
-        if self.strict and self._warnings > 1:
+        if self.strict and self._warnings > 0:
             raise DistutilsSetupError('Please correct your package.')
 
     def check_metadata(self):
diff --git a/Lib/distutils/tests/test_check.py b/Lib/distutils/tests/test_check.py
index 5e0c453140..372bae367b 100644
--- a/Lib/distutils/tests/test_check.py
+++ b/Lib/distutils/tests/test_check.py
@@ -72,17 +72,16 @@ class CheckTestCase(support.LoggingSilencer,
         self.assertEquals(cmd._warnings, 1)
 
         # let's see if we have an error with strict=1
-        cmd = check(dist)
-        cmd.initialize_options()
-        cmd.strict = 1
-        cmd.ensure_finalized()
-        self.assertRaises(DistutilsSetupError, cmd.run)
+        metadata = {'url': 'xxx', 'author': 'xxx',
+                    'author_email': 'xxx',
+                    'name': 'xxx', 'version': 'xxx',
+                    'long_description': broken_rest}
+        self.assertRaises(DistutilsSetupError, self._run, metadata,
+                          **{'strict': 1, 'restructuredtext': 1})
 
         # and non-broken rest
-        rest = 'title\n=====\n\ntest'
-        pkg_info, dist = self.create_dist(long_description=rest)
-        cmd = check(dist)
-        cmd.check_restructuredtext()
+        metadata['long_description'] = 'title\n=====\n\ntest'
+        cmd = self._run(metadata, strict=1, restructuredtext=1)
         self.assertEquals(cmd._warnings, 0)
 
     def test_check_all(self):