In setup.py, inline the packages that need to be installed into setup().

anntzer · anntzer · commit 2256fadf2375 · 2019-05-08T18:00:22.000+02:00
... instead of hiding that behind a function call in setupext.

Also always install the test source files -- setting `tests = False` in
setup.cfg now only ignores the test data (this is also consistent with
the contents of `matplotlib/tests/__init__.py which is clearly intended
to handle the case where the test source files are installed but not the
baseline images).  This is done for simplicity of implementation and
also to make it possibly easier to separately install the baseline
images (as these are now pure "data") in the future.
diff --git a/setup.py b/setup.py
@@ -26,7 +26,7 @@
 import urllib.request
 from zipfile import ZipFile
 
-from setuptools import setup, Extension
+from setuptools import setup, find_packages, Extension
 from setuptools.command.build_ext import build_ext as BuildExtCommand
 from setuptools.command.develop import develop as DevelopCommand
 from setuptools.command.install_lib import install_lib as InstallLibCommand
@@ -182,7 +182,6 @@ def run(self):
     # extensions that can depend on numpy for the build.
     ext_modules = [Extension('', [])]
     package_data = {}
-    package_dir = {'': 'lib'}
     install_requires = []
     setup_requires = []
 
@@ -236,9 +235,6 @@ def run(self):
         # Now collect all of the information we need to build all of the
         # packages.
         for package in good_packages:
-            packages.extend(package.get_packages())
-            namespace_packages.extend(package.get_namespace_packages())
-            py_modules.extend(package.get_py_modules())
             # Extension modules only get added in build_ext, as numpy will have
             # been installed (as setup_requires) at that point.
             data = package.get_package_data()
@@ -275,12 +271,12 @@ def run(self):
         interfaces and hardcopy output formats.
         """,
         license="PSF",
-        packages=packages,
-        namespace_packages=namespace_packages,
-        platforms='any',
-        py_modules=py_modules,
+        platforms="any",
+        package_dir={"": "lib"},
+        packages=find_packages("lib"),
+        namespace_packages=["mpl_toolkits"],
+        py_modules=["pylab"],
         ext_modules=ext_modules,
-        package_dir=package_dir,
         package_data=package_data,
         classifiers=classifiers,
         download_url="https://matplotlib.org/users/installing.html",
diff --git a/setupext.py b/setupext.py
@@ -10,7 +10,6 @@
 import os
 import pathlib
 import platform
-import setuptools
 import shlex
 import shutil
 import subprocess
@@ -313,30 +312,6 @@ def check(self):
         """
         pass
 
-    def get_packages(self):
-        """
-        Get a list of package names to add to the configuration.
-        These are added to the `packages` list passed to
-        `distutils.setup`.
-        """
-        return []
-
-    def get_namespace_packages(self):
-        """
-        Get a list of namespace package names to add to the configuration.
-        These are added to the `namespace_packages` list passed to
-        `distutils.setup`.
-        """
-        return []
-
-    def get_py_modules(self):
-        """
-        Get a list of top-level modules to add to the configuration.
-        These are added to the `py_modules` list passed to
-        `distutils.setup`.
-        """
-        return []
-
     def get_package_data(self):
         """
         Get a package data dictionary to add to the configuration.
@@ -524,15 +499,6 @@ class Matplotlib(SetupPackage):
     def check(self):
         return versioneer.get_version()
 
-    def get_packages(self):
-        return setuptools.find_packages("lib", exclude=["*.tests"])
-
-    def get_namespace_packages(self):
-        return ['mpl_toolkits']
-
-    def get_py_modules(self):
-        return ['pylab']
-
     def get_package_data(self):
         return {
             'matplotlib': [
@@ -572,9 +538,6 @@ class Tests(OptionalPackage):
     name = "tests"
     default_config = False
 
-    def get_packages(self):
-        return setuptools.find_packages("lib", include=["*.tests"])
-
     def get_package_data(self):
         return {
             'matplotlib': [
