[PATCH] gnu: Add python-ply and python2-ply.

[PATCH] gnu: Add python-ply and python2-ply.

[Danny Milosavljevic]

gnu: Add python-ply and python2-ply.
* gnu/packages/python.scm (python-ply, python2-ply): New variables.
---
 gnu/packages/python.scm | 27 +++++++++++++++++++++++++++
 1 file changed, 27 insertions(+)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index dfbf2cc..3ca2a86 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -9009,3 +9009,30 @@ focus on event-based network programming and multiprotocol integration.")
 
 (define-public python2-twisted
   (package-with-python2 python-twisted))
+
+(define-public python-ply
+  (package
+    (name "python-ply")
+    (version "3.8")
+    (source
+      (origin
+        (method url-fetch)
+        (uri (string-append
+               "https://pypi.python.org/packages/96/e0/"
+               "430fcdb6b3ef1ae534d231397bee7e9304be14a47a267e82ebcb3323d0b5/ply-"
+               version
+               ".tar.gz"))
+        (sha256
+          (base32
+            "1f70ipynmiy09k6px2j7v4w5cdrc21za3xs2k6f1bsvb0bzvvlg7"))))
+    (build-system python-build-system)
+    (inputs
+      `(("python-setuptools" ,python-setuptools)))
+    (home-page "http://www.dabeaz.com/ply/")
+    (synopsis "Python Lex & Yacc")
+    (description "ply is an lex/yacc implemented purely in Python.  It uses
+LR parsing and does extensive error checking.")
+    (license bsd-3)))
+
+(define-public python2-ply
+  (package-with-python2 python-ply))

Re: [PATCH] gnu: Add python-ply and python2-ply.

[Leo Famulari]

On Tue, Jun 07, 2016 at 01:01:10AM +0200, Danny Milosavljevic wrote:

Thanks for the patch!

> * gnu/packages/python.scm (python-ply, python2-ply): New variables.

> +        (uri (string-append
> +               "https://pypi.python.org/packages/96/e0/"
> +               "430fcdb6b3ef1ae534d231397bee7e9304be14a47a267e82ebcb3323d0b5/ply-"
> +               version
> +               ".tar.gz"))

We should adapt pypi-uri to handle the new scheme, but in the meantime,
can you put the hash on its own line, to reduce noise in the commit log
for future updates? This would be the whole thing:

                  "96/e0/430fcdb6b3ef1ae534d231397bee7e9304be14a47a267e82ebcb3323d0b5/"

> +    (inputs
> +      `(("python-setuptools" ,python-setuptools)))

Does it use setuptools at run-time or just at build time? If the former,
then it can be a native-input. Also, if it's only required for the
python-2 version, can you use the python2-variant system as shown in the
package definition of python2-pyicu?

> +    (description "ply is an lex/yacc implemented purely in Python.  It uses
> +LR parsing and does extensive error checking.")

`guix lint` prefers to begin the description with a capital 'P'.

Also, it's nice to wrap 'lex' and 'yacc' in Texinfo markup like this:
@code{lex}. Then, the word 'lex' will be rendered specially. I'm not
sure if we have a policy on this but I think it's useful when referring
to other packages.

[PATCHv2] gnu: Add python-ply and python2-ply.

[Danny Milosavljevic]

gnu: Add python-ply and python2-ply.

* gnu/packages/python.scm (python-ply, python2-ply): New variables.
---
 gnu/packages/python.scm | 28 ++++++++++++++++++++++++++++
 1 file changed, 28 insertions(+)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index dfbf2cc..c78577b 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -9009,3 +9009,31 @@ focus on event-based network programming and multiprotocol integration.")
 
 (define-public python2-twisted
   (package-with-python2 python-twisted))
+
+(define-public python-ply
+  (package
+    (name "python-ply")
+    (version "3.8")
+    (source
+      (origin
+        (method url-fetch)
+        (uri (string-append
+               "https://pypi.python.org/packages/"
+               "96/e0/430fcdb6b3ef1ae534d231397bee7e9304be14a47a267e82ebcb3323d0b5"
+               "/ply-" version ".tar.gz"))
+        (sha256
+          (base32
+            "1f70ipynmiy09k6px2j7v4w5cdrc21za3xs2k6f1bsvb0bzvvlg7"))))
+    (build-system python-build-system)
+    (home-page "http://www.dabeaz.com/ply/")
+    (synopsis "Python Lex & Yacc")
+    (description "PLY is an @code{lex}/@code{yacc} implemented purely in Python.
+It uses LR parsing and does extensive error checking.")
+    (license bsd-3)
+    (properties `((python2-variant . ,(delay python2-ply))))))
+
+(define-public python2-ply
+  (package
+    (inherit (package-with-python2
+              (strip-python2-variant python-ply)))
+    (native-inputs `(("python2-setuptools" ,python2-setuptools)))))

Re: [PATCHv2] gnu: Add python-ply and python2-ply.

[Leo Famulari]

On Tue, Jun 07, 2016 at 08:27:10AM +0200, Danny Milosavljevic wrote:
> * gnu/packages/python.scm (python-ply, python2-ply): New variables.

Thanks, applied as d0b6fed6b85!