Always include sample_data in installs.

anntzer · anntzer · commit f1b703a3b758 · 2021-01-27T23:43:10.000+01:00
The whole sample data folder is ~500K now, which is much less than many
compiled extension modules (backend_agg, ft2font, _image, _path, _qhull
shared objects all weight ~2-3M).  Always installing sample_data is a
step towards making Matplotlib installs more homogeneous (i.e., fewer
variants).
diff --git a/setup.cfg.template b/setup.cfg.template
@@ -21,7 +21,6 @@ license_files = LICENSE/*
 # considered optional. All except 'tests' data (meaning the baseline
 # image files) are installed by default, but that can be changed here.
 #tests = False
-#sample_data = True
 
 [gui_support]
 # Matplotlib supports multiple GUI toolkits, known as backends.
diff --git a/setup.py b/setup.py
@@ -61,7 +61,6 @@
     setupext.Platform(),
     setupext.FreeType(),
     setupext.Qhull(),
-    setupext.SampleData(),
     setupext.Tests(),
     setupext.BackendMacOSX(),
     ]
diff --git a/setupext.py b/setupext.py
@@ -365,9 +365,7 @@ def get_package_data(self):
         return {
             'matplotlib': [
                 'mpl-data/matplotlibrc',
-                *_pkg_data_helper('matplotlib', 'mpl-data/fonts'),
-                *_pkg_data_helper('matplotlib', 'mpl-data/images'),
-                *_pkg_data_helper('matplotlib', 'mpl-data/stylelib'),
+                *_pkg_data_helper('matplotlib', 'mpl-data'),
                 *_pkg_data_helper('matplotlib', 'backends/web_backend'),
                 '*.dll',  # Only actually matters on Windows.
             ],
@@ -479,21 +477,6 @@ def get_extensions(self):
         yield ext
 
 
-class SampleData(OptionalPackage):
-    """
-    This handles the sample data that ships with matplotlib.  It is
-    technically optional, though most often will be desired.
-    """
-    name = "sample_data"
-
-    def get_package_data(self):
-        return {
-            'matplotlib': [
-                *_pkg_data_helper('matplotlib', 'mpl-data/sample_data'),
-            ],
-        }
-
-
 class Tests(OptionalPackage):
     name = "tests"
     default_config = False

Original file line number	Diff line number	Diff line change
`@@ -61,7 +61,6 @@`
`61`	`61`	`setupext.Platform(),`
`62`	`62`	`setupext.FreeType(),`
`63`	`63`	`setupext.Qhull(),`
`64`		`- setupext.SampleData(),`
`65`	`64`	`setupext.Tests(),`
`66`	`65`	`setupext.BackendMacOSX(),`
`67`	`66`	`]`