Merge pull request #18895 from timhoffm/python-dep

jklymak · web-flow · commit 294b3f299fe4 · 2020-11-13T13:32:38.000-08:00
Introduce variable since which mpl version the minimal python version
diff --git a/setup.py b/setup.py
@@ -8,17 +8,19 @@
 # and/or pip.
 import sys
 
-min_version = (3, 7)
+py_min_version = (3, 7)  # minimal supported python version
+since_mpl_version = (3, 4)  # py_min_version is required since this mpl version
 
-if sys.version_info < min_version:
+if sys.version_info < py_min_version:
     error = """
-Beginning with Matplotlib 3.4, Python {0} or above is required.
-You are using Python {1}.
+Beginning with Matplotlib {0}, Python {1} or above is required.
+You are using Python {2}.
 
 This may be due to an out of date pip.
 
 Make sure you have pip >= 9.0.1.
-""".format('.'.join(str(n) for n in min_version),
+""".format('.'.join(str(n) for n in since_mpl_version),
+           '.'.join(str(n) for n in py_min_version),
            '.'.join(str(n) for n in sys.version_info[:3]))
     sys.exit(error)
 
@@ -281,7 +283,7 @@ def build_extensions(self):
     ext_modules=[Extension("", [])],
     package_data=package_data,
 
-    python_requires='>={}'.format('.'.join(str(n) for n in min_version)),
+    python_requires='>={}'.format('.'.join(str(n) for n in py_min_version)),
     setup_requires=[
         "certifi>=2020.06.20",
         "numpy>=1.16",
