[Python-checkins] r74164 - in python/branches/py3k: Lib/distutils/extension.py Misc/NEWS

[tarek.ziade]

Author: tarek.ziade
Date: Wed Jul 22 10:57:28 2009
New Revision: 74164

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

........
  r74163 | tarek.ziade | 2009-07-22 10:55:19 +0200 (Wed, 22 Jul 2009) | 1 line
  
  Issue #6545: Removed assert statements in distutils.Extension, so the behavior is similar when used with -O
........


Modified:
   python/branches/py3k/   (props changed)
   python/branches/py3k/Lib/distutils/extension.py
   python/branches/py3k/Misc/NEWS

Modified: python/branches/py3k/Lib/distutils/extension.py
==============================================================================
--- python/branches/py3k/Lib/distutils/extension.py	(original)
+++ python/branches/py3k/Lib/distutils/extension.py	Wed Jul 22 10:57:28 2009
@@ -103,10 +103,11 @@
                   optional=None,
                   **kw                      # To catch unknown keywords
                  ):
-        assert isinstance(name, str), "'name' must be a string"
-        assert (isinstance(sources, list) and
-                all(isinstance(v, str) for v in sources)), \
-                "'sources' must be a list of strings"
+        if not isinstance(name, str):
+            raise AssertionError("'name' must be a string")
+        if not (isinstance(sources, list) and
+                all(isinstance(v, str) for v in sources)):
+            raise AssertionError("'sources' must be a list of strings")
 
         self.name = name
         self.sources = sources

Modified: python/branches/py3k/Misc/NEWS
==============================================================================
--- python/branches/py3k/Misc/NEWS	(original)
+++ python/branches/py3k/Misc/NEWS	Wed Jul 22 10:57:28 2009
@@ -940,6 +940,9 @@
 Library
 -------
 
+- Issue #6545: Removed assert statements in distutils.Extension, so the 
+  behavior is similar when used with -O.
+
 - unittest has been split up into a package.  All old names should still work.
 
 - Issue #6466: now distutils.cygwinccompiler and distutils.emxccompiler