Raise minimum WX version to 2.9.

anntzer · anntzer · commit 4f09f70fe3b2 · 2017-11-05T22:46:54.000-08:00
2.9 was released in 2010; in fact, *before* 2.8.12 (the previous min version), see https://github.com/wxWidgets/wxPython/releases?after=wxPy-2.9.2.1 Remove some conditional code.
diff --git a/INSTALL.rst b/INSTALL.rst
@@ -198,7 +198,7 @@ optional Matplotlib backends and the capabilities they provide.
     `PySide <https://pypi.python.org/pypi/PySide>`_: for the Qt4Agg backend;
   * `PyQt5 <https://pypi.python.org/pypi/PyQt5>`_: for the Qt5Agg backend;
   * :term:`pygtk` (>= 2.4): for the GTK and the GTKAgg backend;
-  * :term:`wxpython` (>= 2.8 or later): for the WX or WXAgg backend;
+  * :term:`wxpython` (>= 2.9 or later): for the WX or WXAgg backend;
   * `pycairo <https://pypi.python.org/pypi/pycairo>`_: for GTK3Cairo;
   * `Tornado <https://pypi.python.org/pypi/tornado>`_: for the WebAgg backend.
 
diff --git a/doc/conf.py b/doc/conf.py
@@ -351,7 +351,7 @@ class ToolBar(object):
     class Frame(object):
         pass
 
-    VERSION_STRING = '2.8.12'
+    VERSION_STRING = '2.9'
 
 
 class MyPyQt4(MagicMock):
diff --git a/lib/matplotlib/backends/backend_wx.py b/lib/matplotlib/backends/backend_wx.py
@@ -684,13 +684,8 @@ def do_nothing(*args, **kwargs):
         self.Bind(wx.EVT_MOUSE_CAPTURE_CHANGED, self._onCaptureLost)
         self.Bind(wx.EVT_MOUSE_CAPTURE_LOST, self._onCaptureLost)
 
-        if wx.VERSION_STRING < "2.9":
-            # only needed in 2.8 to reduce flicker
-            self.SetBackgroundStyle(wx.BG_STYLE_CUSTOM)
-            self.Bind(wx.EVT_ERASE_BACKGROUND, self._onEraseBackground)
-        else:
-            # this does the same in 2.9+
-            self.SetBackgroundStyle(wx.BG_STYLE_PAINT)
+        # Reduce flicker.
+        self.SetBackgroundStyle(wx.BG_STYLE_PAINT)
 
         self.macros = {}  # dict from wx id to seq of macros
 
diff --git a/lib/matplotlib/backends/wx_compat.py b/lib/matplotlib/backends/wx_compat.py
@@ -13,7 +13,7 @@
 import six
 from distutils.version import LooseVersion
 
-missingwx = "Matplotlib backend_wx and backend_wxagg require wxPython >=2.8.12"
+missingwx = "Matplotlib backend_wx and backend_wxagg require wxPython>=2.9"
 
 
 try:
@@ -24,8 +24,7 @@
     raise ImportError(missingwx)
 
 # Ensure we have the correct version imported
-if LooseVersion(wx.VERSION_STRING) < LooseVersion("2.8.12"):
-    print(" wxPython version %s was imported." % backend_version)
+if LooseVersion(wx.VERSION_STRING) < LooseVersion("2.9"):
     raise ImportError(missingwx)
 
 if is_phoenix:
diff --git a/setupext.py b/setupext.py
@@ -1807,7 +1807,7 @@ def check_requirements(self):
                 _wx_ensure_failed = wxversion.VersionError
 
             try:
-                wxversion.ensureMinimal('2.8')
+                wxversion.ensureMinimal('2.9')
             except _wx_ensure_failed:
                 pass
 
@@ -1817,13 +1817,9 @@ def check_requirements(self):
         except ImportError:
             raise CheckFailed("requires wxPython")
 
-        # Extra version check in case wxversion lacks AlreadyImportedError;
-        # then VersionError might have been raised and ignored when
-        # there really *is* a problem with the version.
-        major, minor = [int(n) for n in backend_version.split('.')[:2]]
-        if major < 2 or (major < 3 and minor < 8):
+        if not is_min_version(backend_version, "2.9"):
             raise CheckFailed(
-                "Requires wxPython 2.8, found %s" % backend_version)
+                "Requires wxPython 2.9, found %s" % backend_version)
 
         return "version %s" % backend_version
 
