Issue #27919: Deprecate extra_path option in distutils.

diff --git a/Doc/whatsnew/3.6.rst b/Doc/whatsnew/3.6.rst
index 6ef82d41feecf5985ca084dabc1e7d62f6d09a90..e560fba3378e42a3b45b9617d619695ebb27e1bc 100644 (file)
--- a/Doc/whatsnew/3.6.rst
+++ b/Doc/whatsnew/3.6.rst
@@ -782,6 +782,12 @@ Deprecated features
   now deprecated.
   (Contributed by Serhiy Storchaka in :issue:`25791` and :issue:`26754`.)
 
+* The undocumented ``extra_path`` argument to a distutils Distribution
+  is now considered
+  deprecated, will raise a warning during install if set. Support for this
+  parameter will be dropped in a future Python release and likely earlier
+  through third party tools. See :issue:`27919` for details.
+
 
 Deprecated Python behavior
 --------------------------

diff --git a/Lib/distutils/command/install.py b/Lib/distutils/command/install.py
index fca05d69c619c26a53d02462973fff93a2a9d90a..0258d3deae9467988c42aa71e7750f4b9d526583 100644 (file)
--- a/Lib/distutils/command/install.py
+++ b/Lib/distutils/command/install.py
@@ -175,6 +175,7 @@ class install(Command):
         self.compile = None
         self.optimize = None
 
+        # Deprecated
         # These two are for putting non-packagized distributions into their
         # own directory and creating a .pth file if it makes sense.
         # 'extra_path' comes from the setup file; 'install_path_file' can
@@ -344,6 +345,7 @@ class install(Command):
                            'scripts', 'data', 'headers',
                            'userbase', 'usersite')
 
+        # Deprecated
         # Well, we're not actually fully completely finalized yet: we still
         # have to deal with 'extra_path', which is the hack for allowing
         # non-packagized module distributions (hello, Numerical Python!) to
@@ -490,6 +492,10 @@ class install(Command):
             self.extra_path = self.distribution.extra_path
 
         if self.extra_path is not None:
+            log.warn(
+                "Distribution option extra_path is deprecated. "
+                "See issue27919 for details."
+            )
             if isinstance(self.extra_path, str):
                 self.extra_path = self.extra_path.split(',')
 

diff --git a/Misc/NEWS b/Misc/NEWS
index 83a8b0674dca0edc4adf6ee6e0931eeee34e8fdd..c6d2bafb25592b7d9fc4cae865636c54e22f66a7 100644 (file)
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -60,6 +60,9 @@ Core and Builtins
 Library
 -------
 
+- Issue #27919: Deprecated ``extra_path`` distribution option in distutils
+  packaging.
+
 - Issue #23229: Add new ``cmath`` constants: ``cmath.inf`` and ``cmath.nan`` to
   match ``math.inf`` and ``math.nan``, and also ``cmath.infj`` and
   ``cmath.nanj`` to match the format used by complex repr.