]> granicus.if.org Git - python/commitdiff
Fix a minor inconsistency in capitalization for the 'No module named' exception
authorBrett Cannon <bcannon@gmail.com>
Thu, 18 Nov 2010 03:03:04 +0000 (03:03 +0000)
committerBrett Cannon <bcannon@gmail.com>
Thu, 18 Nov 2010 03:03:04 +0000 (03:03 +0000)
message in importlib.

Thanks to Éric Araujo for spotting the inconsistency.

Lib/importlib/_bootstrap.py
Misc/NEWS

index 1a37a89e886ff84c0db881886321000dc9874a93..9e5455417fdd539fb92e542884a1e1ca8e43e42f 100644 (file)
@@ -758,6 +758,8 @@ class _ImportLockContext:
 
 _IMPLICIT_META_PATH = [BuiltinImporter, FrozenImporter, _DefaultPathFinder]
 
+_ERR_MSG = 'No module named {}'
+
 def _gcd_import(name, package=None, level=0):
     """Import and return the module based on its name, the package the call is
     being made from, and the level adjustment.
@@ -808,8 +810,8 @@ def _gcd_import(name, package=None, level=0):
             try:
                 path = parent_module.__path__
             except AttributeError:
-                raise ImportError("no module named {}; "
-                                    "{} is not a package".format(name, parent))
+                msg = (_ERR_MSG + '; {} is not a package').format(name, parent)
+                raise ImportError(msg)
         meta_path = sys.meta_path + _IMPLICIT_META_PATH
         for finder in meta_path:
             loader = finder.find_module(name, path)
@@ -817,7 +819,7 @@ def _gcd_import(name, package=None, level=0):
                 loader.load_module(name)
                 break
         else:
-            raise ImportError("No module named {0}".format(name))
+            raise ImportError(_ERR_MSG.format(name))
         # Backwards-compatibility; be nicer to skip the dict lookup.
         module = sys.modules[name]
         if parent:
index 13cae784a175aaf5e181d2331706baa071be3dd1..f99919eca4376f2f0545c62f10c7417c5693c58f 100644 (file)
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -18,6 +18,8 @@ Core and Builtins
 Library
 -------
 
+- Make the 'No module named' exception message from importlib consistent.
+
 - Issue #10443: Add the SSLContext.set_default_verify_paths() method.
 
 - Issue #10440: Support RUSAGE_THREAD as a constant in the resource module.