gh-159240: Deprecate __{author,credits,copyright}__ attributes in stdlib modules (#159241)

Author: StanFromIreland

Doc/deprecations/pending-removal-in-3.20.rst

@@ -39,6 +39,27 @@ Pending removal in Python 3.20
 
   (Contributed by Hugo van Kemenade and Stan Ulbrych in :gh:`76007`.)
 
+* The ``__author__``, ``__credits__``, ``__copyright__`` and ``__date__``
+  attributes have been deprecated in these standard library modules and will
+  be removed in Python 3.20. Attribution is preserved in the source code
+  and the version control history.
+
+  - :mod:`bz2`
+  - :mod:`concurrent.futures`
+  - :mod:`inspect`
+  - :mod:`io`
+  - :mod:`json`
+  - :mod:`logging`
+  - :mod:`optparse`
+  - :mod:`platform`
+  - :mod:`pydoc`
+  - :mod:`tarfile`
+  - :mod:`tkinter.ttk`
+  - :mod:`tokenize`
+  - :mod:`uuid`
+
+  (Contributed by Stan Ulbrych in :gh:`159240`.)
+
 * Deprecations defined by :pep:`829`:
 
   * Warnings are produced for ``import`` lines found in :file:`{name}.pth`

Doc/library/test.rst

@@ -1087,6 +1087,14 @@ The :mod:`!test.support` module defines the following functions:
 
    .. versionadded:: 3.6
 
+
+.. function:: check_deprecated_module_attributes(test_case, module, *names, remove)
+
+   Assert that accessing each attribute in *names* on *module* emits a
+   :exc:`DeprecationWarning` naming the attribute and the Python version
+   *remove*, given as a ``(major, minor)`` tuple, in which it will be removed.
+
+
 .. function:: skip_if_broken_multiprocessing_synchronize()
 
    Skip tests if the :mod:`!multiprocessing.synchronize` module is missing, if

Doc/whatsnew/3.16.rst

@@ -1192,6 +1192,29 @@ New deprecations
     3.9, now issues a deprecation warning on use. This property is slated for
     removal in 3.21. Use ``ast.Tuple.elts`` instead.
 
+* ``__author__``, ``__credits__``, ``__copyright__`` and ``__date__``:
+
+  * The ``__author__``, ``__credits__``, ``__copyright__`` and ``__date__``
+    attributes have been deprecated in these standard library modules and will
+    be removed in Python 3.20. Attribution is preserved in the source code
+    and the version control history.
+
+    - :mod:`bz2`
+    - :mod:`concurrent.futures`
+    - :mod:`inspect`
+    - :mod:`io`
+    - :mod:`json`
+    - :mod:`logging`
+    - :mod:`optparse`
+    - :mod:`platform`
+    - :mod:`pydoc`
+    - :mod:`tarfile`
+    - :mod:`tkinter.ttk`
+    - :mod:`tokenize`
+    - :mod:`uuid`
+
+    (Contributed by Stan Ulbrych in :gh:`159240`.)
+
 * :mod:`functools`:
 
   * Using a return annotation to infer the dispatch type for

Lib/bz2.py

@@ -8,12 +8,13 @@
            "open", "compress", "decompress",
            "bzlib_version", "bzlib_version_info"]
 
-__author__ = "Nadeem Vawda <nadeem.vawda@gmail.com>"
+# Original author: Nadeem Vawda <nadeem.vawda@gmail.com>
 
 from builtins import open as _builtin_open
 from compression._common import _streams
 import io
 import os
+lazy import warnings
 
 from _bz2 import (BZ2Compressor, BZ2Decompressor,
                   bzlib_version, bzlib_version_info)
@@ -352,3 +353,10 @@ def decompress(data):
                              "end-of-stream marker was reached")
         data = decomp.unused_data
     return b"".join(results)
+
+
+def __getattr__(name):
+    if name == "__author__":
+        warnings._deprecated("__author__", remove=(3, 20))
+        return "Nadeem Vawda <nadeem.vawda@gmail.com>"  # Do not change
+    raise AttributeError(f"module {__name__!r} has no attribute {name!r}")

Lib/concurrent/futures/__init__.py

@@ -3,7 +3,7 @@
 
 """Execute computations asynchronously using threads or processes."""
 
-__author__ = 'Brian Quinlan (brian@sweetapp.com)'
+# Original author: Brian Quinlan (brian@sweetapp.com)
 
 from concurrent.futures._base import (FIRST_COMPLETED,
                                       FIRST_EXCEPTION,
@@ -17,6 +17,7 @@
                                       wait,
                                       as_completed)
 
+lazy import warnings
 lazy from .process import ProcessPoolExecutor
 lazy from .thread import ThreadPoolExecutor
 
@@ -48,4 +49,11 @@
 
 
 def __dir__():
-    return __all__ + ['__author__', '__doc__']
+    return __all__ + ['__doc__']
+
+
+def __getattr__(name):
+    if name == "__author__":
+        warnings._deprecated("__author__", remove=(3, 20))
+        return 'Brian Quinlan (brian@sweetapp.com)'  # Do not change
+    raise AttributeError(f"module {__name__!r} has no attribute {name!r}")

Lib/concurrent/futures/_base.py

@@ -1,7 +1,7 @@
 # Copyright 2009 Brian Quinlan. All Rights Reserved.
 # Licensed to PSF under a Contributor Agreement.
 
-__author__ = 'Brian Quinlan (brian@sweetapp.com)'
+# Original author: Brian Quinlan (brian@sweetapp.com)
 
 import collections
 import logging

Lib/concurrent/futures/process.py

@@ -43,7 +43,7 @@
   _ResultItems in "Result Q"
 """
 
-__author__ = 'Brian Quinlan (brian@sweetapp.com)'
+# Original author: Brian Quinlan (brian@sweetapp.com)
 
 import os
 from concurrent.futures import _base
@@ -58,6 +58,7 @@
 from functools import partial
 import itertools
 import sys
+lazy import warnings
 from traceback import format_exception
 
 
@@ -1065,3 +1066,10 @@ def kill_workers(self):
         submitted).
         """
         return self._force_shutdown(operation=_KILL)
+
+
+def __getattr__(name):
+    if name == "__author__":
+        warnings._deprecated("__author__", remove=(3, 20))
+        return 'Brian Quinlan (brian@sweetapp.com)'  # Do not change
+    raise AttributeError(f"module {__name__!r} has no attribute {name!r}")

Lib/concurrent/futures/thread.py

@@ -3,7 +3,7 @@
 
 """Implements ThreadPoolExecutor."""
 
-__author__ = 'Brian Quinlan (brian@sweetapp.com)'
+# Original author: Brian Quinlan (brian@sweetapp.com)
 
 from concurrent.futures import _base
 import itertools
@@ -12,6 +12,7 @@
 import types
 import weakref
 import os
+lazy import warnings
 
 
 _threads_queues = weakref.WeakKeyDictionary()
@@ -272,3 +273,10 @@ def shutdown(self, wait=True, *, cancel_futures=False):
             for t in self._threads:
                 t.join()
     shutdown.__doc__ = _base.Executor.shutdown.__doc__
+
+
+def __getattr__(name):
+    if name == "__author__":
+        warnings._deprecated("__author__", remove=(3, 20))
+        return 'Brian Quinlan (brian@sweetapp.com)'  # Do not change
+    raise AttributeError(f"module {__name__!r} has no attribute {name!r}")

Lib/ctypes/_aix.py

@@ -44,7 +44,7 @@
 The mode bit RTLD_MEMBER tells initAndLoad() that it needs to use the AIX (SVR3)
 naming style.
 """
-__author__ = "Michael Felt <aixtools@felt.demon.nl>"
+# Original author: Michael Felt <aixtools@felt.demon.nl>
 
 import re
 from os import environ, path

Lib/inspect.py

@@ -28,8 +28,8 @@
 
 # This module is in the public domain.  No warranties.
 
-__author__ = ('Ka-Ping Yee <ping@lfw.org>',
-              'Yury Selivanov <yselivanov@sprymix.com>')
+# Original authors: Ka-Ping Yee,
+#                   Yury Selivanov <yselivanov@sprymix.com>
 
 __all__ = [
     "AGEN_CLOSED",
@@ -160,6 +160,7 @@
 import types
 import functools
 import builtins
+lazy import warnings
 from keyword import iskeyword
 from operator import attrgetter
 from collections import namedtuple, OrderedDict
@@ -3540,5 +3541,13 @@ def _main():
             print(getsource(obj))
 
 
+def __getattr__(name):
+    if name == "__author__":
+        warnings._deprecated("__author__", remove=(3, 20))
+        return ('Ka-Ping Yee',
+                'Yury Selivanov <yselivanov@sprymix.com>')  # Do not change
+    raise AttributeError(f"module {__name__!r} has no attribute {name!r}")
+
+
 if __name__ == "__main__":
     _main()