Backport PR #16394: Mark inkscape 1.0 as unsupported (at least for now).

tacaswell · meeseeksmachine · commit 4fe1e3baa96d · 2020-02-03T16:41:03.000Z
diff --git a/doc/devel/testing.rst b/doc/devel/testing.rst
@@ -28,7 +28,7 @@ The following software is required to run the tests:
 
 - pytest_ (>=3.6)
 - Ghostscript_ (>= 9.0, to render PDF files)
-- Inkscape_ (to render SVG files)
+- Inkscape_ (<1.0, to render SVG files)
 
 Optionally you can install:
 
diff --git a/lib/matplotlib/__init__.py b/lib/matplotlib/__init__.py
@@ -345,7 +345,7 @@ def impl(args, regex, min_ver=None, ignore_exit_code=False):
             if min_ver is not None and version < min_ver:
                 raise ExecutableNotFoundError(
                     f"You have {args[0]} version {version} but the minimum "
-                    f"version supported by Matplotlib is {min_ver}.")
+                    f"version supported by Matplotlib is {min_ver}")
             return _ExecInfo(args[0], version)
         else:
             raise ExecutableNotFoundError(
@@ -366,7 +366,12 @@ def impl(args, regex, min_ver=None, ignore_exit_code=False):
         message = "Failed to find a Ghostscript installation"
         raise ExecutableNotFoundError(message)
     elif name == "inkscape":
-        return impl(["inkscape", "-V"], "^Inkscape ([^ ]*)")
+        info = impl(["inkscape", "-V"], "^Inkscape ([^ ]*)")
+        if info and info.version >= "1.0":
+            raise ExecutableNotFoundError(
+                f"You have Inkscape version {info.version} but Matplotlib "
+                f"only supports Inkscape<1.0")
+        return info
     elif name == "magick":
         path = None
         if sys.platform == "win32":
@@ -403,7 +408,7 @@ def impl(args, regex, min_ver=None, ignore_exit_code=False):
                          or "0.9" <= info.version <= "1.0"):
             raise ExecutableNotFoundError(
                 f"You have pdftops version {info.version} but the minimum "
-                f"version supported by Matplotlib is 3.0.")
+                f"version supported by Matplotlib is 3.0")
         return info
     else:
         raise ValueError("Unknown executable: {!r}".format(name))
