unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#33936] Add python-pygraphviz.
@ 2018-12-31 13:40 Ricardo Wurmus
  2018-12-31 13:48 ` [bug#33936] [PATCH 1/3] gnu: Add graphviz-2.38 Ricardo Wurmus
  2019-01-12 10:44 ` [bug#33936] Add python-pygraphviz Ludovic Courtès
  0 siblings, 2 replies; 6+ messages in thread
From: Ricardo Wurmus @ 2018-12-31 13:40 UTC (permalink / raw)
  To: 33936

Hi Guix,

this patch series adds python-pygraphviz.  Unfortunately, it depends on
a previous release of GraphViz due to a bug in the latest release.  We
can drop it with the next release of GraphViz.

--
Ricardo

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [bug#33936] [PATCH 1/3] gnu: Add graphviz-2.38.
  2018-12-31 13:40 [bug#33936] Add python-pygraphviz Ricardo Wurmus
@ 2018-12-31 13:48 ` Ricardo Wurmus
  2018-12-31 13:48   ` [bug#33936] [PATCH 2/3] gnu: Add python-doctest-ignore-unicode Ricardo Wurmus
  2018-12-31 13:48   ` [bug#33936] [PATCH 3/3] gnu: Add python-graphviz Ricardo Wurmus
  2019-01-12 10:44 ` [bug#33936] Add python-pygraphviz Ludovic Courtès
  1 sibling, 2 replies; 6+ messages in thread
From: Ricardo Wurmus @ 2018-12-31 13:48 UTC (permalink / raw)
  To: 33936; +Cc: Ricardo Wurmus

* gnu/packages/graphviz.scm (graphviz-2.38): New variable.
---
 gnu/packages/graphviz.scm | 51 ++++++++++++++++++++++++++++++++++++++-
 1 file changed, 50 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/graphviz.scm b/gnu/packages/graphviz.scm
index 569923d4e..47342b784 100644
--- a/gnu/packages/graphviz.scm
+++ b/gnu/packages/graphviz.scm
@@ -2,7 +2,7 @@
 ;;; Copyright © 2013, 2015 Ludovic Courtès <ludo@gnu.org>
 ;;; Copyright © 2015 Efraim Flashner <efraim@flashner.co.il>
 ;;; Copyright © 2016 Theodoros Foradis <theodoros@foradis.org>
-;;; Copyright © 2017 Ricardo Wurmus <rekado@elephly.net>
+;;; Copyright © 2017, 2018 Ricardo Wurmus <rekado@elephly.net>
 ;;; Copyright © 2017, 2018 Tobias Geerinckx-Rice <me@tobias.gr>
 ;;; Copyright © 2017 Gábor Boskovits <boskovits@gmail.com>
 ;;; Copyright © 2018 Mathieu Lirzin <mthl@gnu.org>
@@ -27,7 +27,10 @@
   #:use-module (guix build-system gnu)
   #:use-module (guix build-system python)
   #:use-module (guix download)
+  #:use-module (guix git-download)
+  #:use-module (guix utils)
   #:use-module (gnu packages xorg)
+  #:use-module (gnu packages flex)
   #:use-module (gnu packages gtk)
   #:use-module (gnu packages xml)
   #:use-module (gnu packages glib)
@@ -35,6 +38,7 @@
   #:use-module (gnu packages bison)
   #:use-module (gnu packages image)
   #:use-module (gnu packages autotools)
+  #:use-module (gnu packages perl)
   #:use-module (gnu packages pkg-config)
   #:use-module (gnu packages glib)
   #:use-module (gnu packages gtk)
@@ -43,6 +47,7 @@
   #:use-module (gnu packages compression)
   #:use-module (gnu packages gd)
   #:use-module (gnu packages swig)
+  #:use-module (gnu packages tcl)
   #:use-module (gnu packages python)
   #:use-module (gnu packages tex)
   #:use-module ((guix licenses) #:prefix license:))
@@ -114,6 +119,50 @@ software engineering, database and web design, machine learning, and in visual
 interfaces for other technical domains.")
     (license license:epl1.0)))
 
+;; Older Graphviz needed for pygraphviz.  See
+;; https://github.com/pygraphviz/pygraphviz/issues/175
+(define-public graphviz-2.38
+  ;; This commit corresponds to the changelog change for version 2.38.0.
+  ;; There are no tags.
+  (let ((commit "f54ac2c9313ae80ccf76ef4ac6aa9be820a23126")
+        (revision "1"))
+    (package (inherit graphviz)
+      (name "graphviz")
+      (version (git-version "2.38.0" revision commit))
+      (source (origin
+                (method git-fetch)
+                (uri (git-reference
+                      (url "https://gitlab.com/graphviz/graphviz.git")
+                      (commit commit)))
+                (file-name (git-file-name name version))
+                (sha256
+                 (base32
+                  "1vjg308gflmi1khgjmcj431cnkrlv12bg4cqah39mwhny92jy92x"))))
+      (arguments
+       (substitute-keyword-arguments (package-arguments graphviz)
+         ((#:phases phases)
+          `(modify-phases ,phases
+             (add-after 'unpack 'prepare-bootstrap
+               (lambda _
+                 (substitute* "autogen.sh"
+                   (("/bin/sh") (which "sh"))
+                   (("\\$GRAPHVIZ_VERSION_DATE") "0"))
+                 (setenv "CONFIG_SHELL" (which "sh"))
+                 (setenv "SHELL" (which "sh"))
+
+                 (map make-file-writable (find-files "." ".*"))
+                 #t))
+             (replace 'bootstrap
+               (lambda _ (invoke (which "sh") "autogen.sh" "NOCONFIG") #t))))))
+      (native-inputs
+       `(("autoconf" ,autoconf)
+         ("automake" ,automake)
+         ("libtool" ,libtool)
+         ("flex" ,flex)
+         ("perl" ,perl)
+         ("tcl" ,tcl)
+         ,@(package-native-inputs graphviz))))))
+
 (define-public python-graphviz
   (package
     (name "python-graphviz")
-- 
2.19.1

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [bug#33936] [PATCH 2/3] gnu: Add python-doctest-ignore-unicode.
  2018-12-31 13:48 ` [bug#33936] [PATCH 1/3] gnu: Add graphviz-2.38 Ricardo Wurmus
@ 2018-12-31 13:48   ` Ricardo Wurmus
  2018-12-31 13:48   ` [bug#33936] [PATCH 3/3] gnu: Add python-graphviz Ricardo Wurmus
  1 sibling, 0 replies; 6+ messages in thread
From: Ricardo Wurmus @ 2018-12-31 13:48 UTC (permalink / raw)
  To: 33936; +Cc: Ricardo Wurmus

* gnu/packages/python.scm (python-doctest-ignore-unicode): New variable.
---
 gnu/packages/python.scm | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index bb4c5ab76..81fbe2b68 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -14980,6 +14980,27 @@ RFC 8265 and RFC 8266.")
     (properties '((upstream-name . "Wikidata")))
     (license license:gpl3+)))
 
+(define-public python-doctest-ignore-unicode
+  (package
+    (name "python-doctest-ignore-unicode")
+    (version "0.1.2")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (pypi-uri "doctest-ignore-unicode" version))
+       (sha256
+        (base32
+         "1m9aa4qnyj21lbq4sbvmv1vcz7zksss4rz37ddf2hxv4hk8b547w"))))
+    (build-system python-build-system)
+    (native-inputs
+     `(("python-nose" ,python-nose)))
+    (home-page "https://github.com/gnublade/doctest-ignore-unicode")
+    (synopsis "Ignore Unicode literal prefixes in doctests")
+    (description
+     "This package adds support for a flag to ignore Unicode literal prefixes
+in doctests.")
+    (license license:asl2.0)))
+
 (define-public python-attr
   (package
     (name "python-attr")
-- 
2.19.1

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [bug#33936] [PATCH 3/3] gnu: Add python-graphviz.
  2018-12-31 13:48 ` [bug#33936] [PATCH 1/3] gnu: Add graphviz-2.38 Ricardo Wurmus
  2018-12-31 13:48   ` [bug#33936] [PATCH 2/3] gnu: Add python-doctest-ignore-unicode Ricardo Wurmus
@ 2018-12-31 13:48   ` Ricardo Wurmus
  1 sibling, 0 replies; 6+ messages in thread
From: Ricardo Wurmus @ 2018-12-31 13:48 UTC (permalink / raw)
  To: 33936; +Cc: Ricardo Wurmus

* gnu/packages/graphviz.scm (python-graphviz): New variable.
---
 gnu/packages/graphviz.scm | 35 +++++++++++++++++++++++++++++++++++
 1 file changed, 35 insertions(+)

diff --git a/gnu/packages/graphviz.scm b/gnu/packages/graphviz.scm
index 47342b784..42f793b48 100644
--- a/gnu/packages/graphviz.scm
+++ b/gnu/packages/graphviz.scm
@@ -29,6 +29,7 @@
   #:use-module (guix download)
   #:use-module (guix git-download)
   #:use-module (guix utils)
+  #:use-module (gnu packages check)
   #:use-module (gnu packages xorg)
   #:use-module (gnu packages flex)
   #:use-module (gnu packages gtk)
@@ -186,6 +187,40 @@ visualization tool suite.")
 (define-public python2-graphviz
   (package-with-python2 python-graphviz))
 
+(define-public python-pygraphviz
+  (package
+    (name "python-pygraphviz")
+    (version "1.5")
+    (source
+     (origin
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://github.com/pygraphviz/pygraphviz.git")
+             (commit (string-append "pygraphviz-" version))))
+       (file-name (string-append "pygraphviz-" version "-checkout"))
+       (sha256
+        (base32
+         "1yldym38m8ckgflln83i88143pd9fjj1vfp23sq39fs6np5g0nzp"))))
+    (build-system python-build-system)
+    (arguments
+     `(#:configure-flags
+       (let ((graphviz (assoc-ref %build-inputs "graphviz")))
+         (list (string-append "--include-path=" graphviz "/include")
+               (string-append "--library-path=" graphviz "/lib")))))
+    (inputs
+     `(("graphviz" ,graphviz-2.38)))
+    (native-inputs
+     `(("python-nose" ,python-nose)
+       ("python-mock" ,python-mock)
+       ("python-doctest-ignore-unicode" ,python-doctest-ignore-unicode)))
+    (home-page "http://pygraphviz.github.io")
+    (synopsis "Python interface to Graphviz")
+    (description "PyGraphviz is a Python interface to the Graphviz graph
+layout and visualization package.  With PyGraphviz you can create, edit, read,
+write, and draw graphs using Python to access the Graphviz graph data
+structure and layout algorithms.")
+    (license license:bsd-3)))
+
 (define-public gts
   (package
     (name "gts")
-- 
2.19.1

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [bug#33936] Add python-pygraphviz.
  2018-12-31 13:40 [bug#33936] Add python-pygraphviz Ricardo Wurmus
  2018-12-31 13:48 ` [bug#33936] [PATCH 1/3] gnu: Add graphviz-2.38 Ricardo Wurmus
@ 2019-01-12 10:44 ` Ludovic Courtès
  2019-01-13  9:08   ` bug#33936: " Ricardo Wurmus
  1 sibling, 1 reply; 6+ messages in thread
From: Ludovic Courtès @ 2019-01-12 10:44 UTC (permalink / raw)
  To: Ricardo Wurmus; +Cc: 33936

Hi,

Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de> skribis:

> this patch series adds python-pygraphviz.  Unfortunately, it depends on
> a previous release of GraphViz due to a bug in the latest release.  We
> can drop it with the next release of GraphViz.

All three patch LGTM, thanks!

Ludo’.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#33936: Add python-pygraphviz.
  2019-01-12 10:44 ` [bug#33936] Add python-pygraphviz Ludovic Courtès
@ 2019-01-13  9:08   ` Ricardo Wurmus
  0 siblings, 0 replies; 6+ messages in thread
From: Ricardo Wurmus @ 2019-01-13  9:08 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 33936-done


Ludovic Courtès <ludo@gnu.org> writes:

> Hi,
>
> Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de> skribis:
>
>> this patch series adds python-pygraphviz.  Unfortunately, it depends on
>> a previous release of GraphViz due to a bug in the latest release.  We
>> can drop it with the next release of GraphViz.
>
> All three patch LGTM, thanks!

Thanks.  Pushed to the master branch after fixing the commit message for
python-pygraphviz.

--
Ricardo

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2019-01-13  9:10 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-12-31 13:40 [bug#33936] Add python-pygraphviz Ricardo Wurmus
2018-12-31 13:48 ` [bug#33936] [PATCH 1/3] gnu: Add graphviz-2.38 Ricardo Wurmus
2018-12-31 13:48   ` [bug#33936] [PATCH 2/3] gnu: Add python-doctest-ignore-unicode Ricardo Wurmus
2018-12-31 13:48   ` [bug#33936] [PATCH 3/3] gnu: Add python-graphviz Ricardo Wurmus
2019-01-12 10:44 ` [bug#33936] Add python-pygraphviz Ludovic Courtès
2019-01-13  9:08   ` bug#33936: " Ricardo Wurmus

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).