bug#29144: Updated patch

[Marius Bakke]

Gábor Boskovits <address@hidden> writes:

> I've sent an update patch with the requested modidications modulo the unit
> tests.
>
> It seems, that unit tests are not included in the pypi tarball.
>
> Should we go with a git checkout, and do unit tests, or should we leave it
> this way?

Bah.  It would be good to have unit tests, but I don't want to hold this
patch back, so I committed it with the following changes for now:

diff --git a/gnu/packages/graphviz.scm b/gnu/packages/graphviz.scm
index dbc501733..2fefb41c0 100644
--- a/gnu/packages/graphviz.scm
+++ b/gnu/packages/graphviz.scm
@@ -226,14 +226,16 @@ be used either as a standalone application, or as a 
python library.")
         (base32
          "00imlz0033dygb9gdag1xr0cybn33gk5jsdi9ffbszzr97rd7dgd"))))
     (build-system python-build-system)
+    ;; FIXME: No tests in PyPi release tarball.
+    (arguments '(#:tests? #f))
     (propagated-inputs
-     `(("python-pyparsing",python-pyparsing)))
-    (native-inputs
-     `(("python-chardet",python-chardet)))
+     `(("python-pyparsing" ,python-pyparsing)))
     (home-page "https://github.com/erocarrera/pydot";)
-    (synopsis "Python interface to Graphviz's Dot language")
+    (synopsis "Python interface to Graphviz's DOT language")
     (description
-     "An interface to Graphviz that can parse and dump
-+into the DOT language used by GraphViz, is written in pure Python,
-+and networkx can convert its graphs to pydot.")
+     "Pydot provides an interface to create, handle, modify and process
+graphs in Graphviz's DOT language, written in pure Python.")
     (license license:expat)))
+
+(define-public python2-pydot
+  (package-with-python2 python-pydot))

Sorry for rewriting the description, but I thought the double Graphviz
mention was redundant and the NetworkX reference rather "out of place"
(presumably other tools too can output to Pydot).  Since the original
text was copied off the README I hope it was okay.  Improvements welcome!

Also adjusted the commit message to mention the extra variable.  Thanks!

signature.asc
Description: PGP signature