Issue #13959: Document imp.find_module/load_module as deprecated.

brettcannon · brettcannon · commit 62961dde319e · 2012-05-13T13:04:21.000-04:00
The code itself does not raise a DeprecationWarning as the functions
are technically fine, it's just a bad API. Unfortunately experience
has shown that the terrible API has been exposed in various places,
necessitating that it stick around probably until py4k comes around
since it is such a shift to move over to importlib.find_loader().
diff --git a/Doc/library/imp.rst b/Doc/library/imp.rst
@@ -72,6 +72,9 @@ This module provides an interface to the mechanisms used to implement the
    then use :func:`find_module` with the *path* argument set to ``P.__path__``.
    When *P* itself has a dotted name, apply this recipe recursively.
 
+   .. deprecated:: 3.3
+      Use :func:`importlib.find_loader` instead.
+
 
 .. function:: load_module(name, file, pathname, description)
 
@@ -93,6 +96,10 @@ This module provides an interface to the mechanisms used to implement the
    it was not ``None``, even when an exception is raised.  This is best done
    using a :keyword:`try` ... :keyword:`finally` statement.
 
+   .. deprecated:: 3.3
+      Unneeded as loaders should be used to load modules and
+      :func:`find_module` is deprecated.
+
 
 .. function:: new_module(name)
 
diff --git a/Misc/NEWS b/Misc/NEWS
@@ -26,7 +26,8 @@ Library
 - Issue #14366: Support lzma compression in zip files.
   Patch by Serhiy Storchaka.
 
-- Issue #13959: Introduce importlib.find_loader().
+- Issue #13959: Introduce importlib.find_loader() and document
+  imp.find_module/load_module as deprecated.
 
 - Issue #14082: shutil.copy2() now copies extended attributes, if possible.
   Patch by Hynek Schlawack.
