diff mbox

gnu: Add python-ply and python2-ply.

Message ID 20160607010110.52922ed9@scratchpost.org
State New
Headers show

Commit Message

Danny Milosavljevic June 6, 2016, 11:01 p.m. UTC
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(+)

Comments

Leo Famulari June 7, 2016, 2:26 a.m. UTC | #1
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.
diff mbox

Patch

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))