unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#38554] [PATCH 0/5]  Update python-hy to 0.17.0
@ 2019-12-10  2:29 Jesse Gibbons
  2019-12-10  2:35 ` [bug#38554] [PATCH 1/5] Add python-fastentrypoints Jesse Gibbons
                   ` (8 more replies)
  0 siblings, 9 replies; 15+ messages in thread
From: Jesse Gibbons @ 2019-12-10  2:29 UTC (permalink / raw)
  To: 38554

The next five patches:
- add dependencies for python-hy 0.17.0 (patches 1,2)
- update python-hy from 0.13.0 to 0.17.0 (patch 3)
- add dependencies for python2-hy 0.17.0 (patches 4,5)


This will be the last version of python-hy to support python 2. If it is
better to drop python2-hy now, simply omit patches 4 and 5, and remove
python2-hy.

-- 
-Jesse

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

* [bug#38554] [PATCH 1/5] Add python-fastentrypoints
  2019-12-10  2:29 [bug#38554] [PATCH 0/5] Update python-hy to 0.17.0 Jesse Gibbons
@ 2019-12-10  2:35 ` Jesse Gibbons
  2019-12-10  2:36 ` [bug#38554] [PATCH 2/5] Add python-funcparserlib Jesse Gibbons
                   ` (7 subsequent siblings)
  8 siblings, 0 replies; 15+ messages in thread
From: Jesse Gibbons @ 2019-12-10  2:35 UTC (permalink / raw)
  To: 38554

[-- Attachment #1: Type: text/plain, Size: 3 bytes --]




[-- Attachment #2: 0001-gnu-add-python-fastentrypoints.patch --]
[-- Type: text/x-patch, Size: 1787 bytes --]

From 97edcfcbe08031240cc7bae12d588695ff5b81ca Mon Sep 17 00:00:00 2001
From: Jesse Gibbons <jgibbons2357+guix@gmail.com>
Date: Mon, 9 Dec 2019 18:16:18 -0700
Subject: [PATCH 1/5] gnu: add python-fastentrypoints

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

diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index 186e558f2f..7026e65ab6 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -67,6 +67,7 @@
 ;;; Copyright © 2019 Giacomo Leidi <goodoldpaul@autistici.org>
 ;;; Copyright © 2019 Wiktor Żelazny <wzelazny@vurv.cz>
 ;;; Copyright © 2019 Tanguy Le Carrour <tanguy@bioneland.org>
+;;; Copyright © 2019 Jesse Gibbons <jgibbons2357+guix@gmail.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -16775,3 +16776,25 @@ scratch and manipulate data from Intel HEX file format.  It also includes
 several convenience Python scripts, including \"classic\" hex2bin and bin2hex
 converters and more, those based on the library itself.")
     (license license:bsd-3)))
+
+
+(define-public python-fastentrypoints
+  (package
+    (name "python-fastentrypoints")
+    (version "0.12")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (pypi-uri "fastentrypoints" version))
+       (sha256
+        (base32
+         "02s1j8i2dzbpbwgq2a3fiqwm3cnmhii2qzc0k42l0rdxd4a4ya7z"))))
+    (build-system python-build-system)
+    (home-page
+     "https://github.com/ninjaaron/fast-entry_points")
+    (synopsis
+     "Makes entry_points specified in setup.py load more quickly")
+    (description
+     "Makes entry_points specified in setup.py load more quickly")
+    (license license:bsd-3)))
+
-- 
2.24.0


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

* [bug#38554] [PATCH 2/5]  Add python-funcparserlib
  2019-12-10  2:29 [bug#38554] [PATCH 0/5] Update python-hy to 0.17.0 Jesse Gibbons
  2019-12-10  2:35 ` [bug#38554] [PATCH 1/5] Add python-fastentrypoints Jesse Gibbons
@ 2019-12-10  2:36 ` Jesse Gibbons
  2019-12-10  2:37 ` [bug#38554] [PATCH 3/5] Update python-hy to 0.17.0 Jesse Gibbons
                   ` (6 subsequent siblings)
  8 siblings, 0 replies; 15+ messages in thread
From: Jesse Gibbons @ 2019-12-10  2:36 UTC (permalink / raw)
  To: 38554

[-- Attachment #1: Type: text/plain, Size: 3 bytes --]




[-- Attachment #2: 0002-gnu-Add-python-funcparserlib.patch --]
[-- Type: text/x-patch, Size: 1598 bytes --]

From 812077cee02fc4e0f50b69810209b1b40e60dfc0 Mon Sep 17 00:00:00 2001
From: Jesse Gibbons <jgibbons2357+guix@gmail.com>
Date: Mon, 9 Dec 2019 18:31:12 -0700
Subject: [PATCH 2/5] gnu: Add python-funcparserlib

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

diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index 7026e65ab6..2161caa2c5 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -16798,3 +16798,30 @@ converters and more, those based on the library itself.")
      "Makes entry_points specified in setup.py load more quickly")
     (license license:bsd-3)))
 
+(define-public python-funcparserlib
+  (package
+    (name "python-funcparserlib")
+    (version "0.3.6")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (pypi-uri "funcparserlib" version))
+       (sha256
+        (base32
+         "07f9cgjr3h4j2m67fhwapn8fja87vazl58zsj4yppf9y3an2x6dp"))))
+    (native-inputs
+     `(("python-tox" ,python-tox)))
+    (arguments
+     `(#:phases
+       (modify-phases %standard-phases
+         (replace 'check
+           (lambda _
+             (invoke "tox"))))))
+    (build-system python-build-system)
+    (home-page
+     "https://github.com/vlasovskikh/funcparserlib")
+    (synopsis
+     "Recursive descent parsing library based on functional combinators")
+    (description
+     "Recursive descent parsing library based on functional combinators")
+    (license license:expat)))
-- 
2.24.0


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

* [bug#38554] [PATCH 3/5] Update python-hy to 0.17.0
  2019-12-10  2:29 [bug#38554] [PATCH 0/5] Update python-hy to 0.17.0 Jesse Gibbons
  2019-12-10  2:35 ` [bug#38554] [PATCH 1/5] Add python-fastentrypoints Jesse Gibbons
  2019-12-10  2:36 ` [bug#38554] [PATCH 2/5] Add python-funcparserlib Jesse Gibbons
@ 2019-12-10  2:37 ` Jesse Gibbons
  2019-12-10  2:37 ` [bug#38554] [PATCH 4/5] Add python2-fastentrypoints Jesse Gibbons
                   ` (5 subsequent siblings)
  8 siblings, 0 replies; 15+ messages in thread
From: Jesse Gibbons @ 2019-12-10  2:37 UTC (permalink / raw)
  To: 38554

[-- Attachment #1: Type: text/plain, Size: 3 bytes --]




[-- Attachment #2: 0003-gnu-python-hy-Update-to-0.17.0.patch --]
[-- Type: text/x-patch, Size: 1642 bytes --]

From 13ac60ca6f9037b9a49a76d2e1bd4b232be36106 Mon Sep 17 00:00:00 2001
From: Jesse Gibbons <jgibbons2357+guix@gmail.com>
Date: Mon, 9 Dec 2019 18:58:37 -0700
Subject: [PATCH 3/5] gnu: python-hy: Update to 0.17.0

* gnu/packages/python-xyz.scm (python-hy): Update to 0.17.0
[inputs]: Add python-fastentrypoints.
[inputs]: Add python-funcparserlib.
---
 gnu/packages/python-xyz.scm | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index 2161caa2c5..6bee2fdd37 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -8600,13 +8600,13 @@ with a new public API, and RPython support.")
 (define-public python-hy
   (package
     (name "python-hy")
-    (version "0.13.0")
+    (version "0.17.0")
     (source (origin
               (method url-fetch)
               (uri (pypi-uri "hy" version))
               (sha256
                (base32
-                "19sfymaksx9jhksfnb15ahid46mzrhdfzz6yy2craz2qnzvpmky8"))))
+                "1gdbqsirsdxj320wnp7my5awzs1kfs6m4fqmkzbd1zd47qzj0zfi"))))
     (build-system python-build-system)
     (arguments
      '(#:phases
@@ -8625,7 +8625,9 @@ with a new public API, and RPython support.")
     (propagated-inputs
      `(("python-astor" ,python-astor)
        ("python-clint" ,python-clint)
-       ("python-rply" ,python-rply)))
+       ("python-rply" ,python-rply)
+       ("python-fastentrypoints" ,python-fastentrypoints)
+       ("python-funcparserlib" ,python-funcparserlib)))
     (home-page "http://hylang.org/")
     (synopsis "Lisp frontend to Python")
     (description
-- 
2.24.0


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

* [bug#38554] [PATCH 4/5]  Add python2-fastentrypoints
  2019-12-10  2:29 [bug#38554] [PATCH 0/5] Update python-hy to 0.17.0 Jesse Gibbons
                   ` (2 preceding siblings ...)
  2019-12-10  2:37 ` [bug#38554] [PATCH 3/5] Update python-hy to 0.17.0 Jesse Gibbons
@ 2019-12-10  2:37 ` Jesse Gibbons
  2019-12-12 17:18   ` zimoun
  2019-12-10  2:38 ` [bug#38554] [PATCH 5/5] Add python2-funcparserlib Jesse Gibbons
                   ` (4 subsequent siblings)
  8 siblings, 1 reply; 15+ messages in thread
From: Jesse Gibbons @ 2019-12-10  2:37 UTC (permalink / raw)
  To: 38554

[-- Attachment #1: Type: text/plain, Size: 3 bytes --]




[-- Attachment #2: 0004-gnu-Add-python2-fastentrypoints.patch --]
[-- Type: text/x-patch, Size: 905 bytes --]

From c93b60f2ea429657a53cc7c018443b1036cdc3fe Mon Sep 17 00:00:00 2001
From: Jesse Gibbons <jgibbons2357+guix@gmail.com>
Date: Mon, 9 Dec 2019 19:05:17 -0700
Subject: [PATCH 4/5] gnu: Add python2-fastentrypoints

* gnu/packages/python-xyz.scm (python2-fastentrypoints): New variable.
---
 gnu/packages/python-xyz.scm | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index 6bee2fdd37..9c476b0bce 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -16800,6 +16800,9 @@ converters and more, those based on the library itself.")
      "Makes entry_points specified in setup.py load more quickly")
     (license license:bsd-3)))
 
+(define-public python2-fastentrypoints
+  (package-with-python2 python-fastentrypoints))
+
 (define-public python-funcparserlib
   (package
     (name "python-funcparserlib")
-- 
2.24.0


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

* [bug#38554] [PATCH 5/5]  Add python2-funcparserlib
  2019-12-10  2:29 [bug#38554] [PATCH 0/5] Update python-hy to 0.17.0 Jesse Gibbons
                   ` (3 preceding siblings ...)
  2019-12-10  2:37 ` [bug#38554] [PATCH 4/5] Add python2-fastentrypoints Jesse Gibbons
@ 2019-12-10  2:38 ` Jesse Gibbons
  2019-12-10  4:25 ` [bug#38554] [PATCH 0/5] Update python-hy to 0.17.0 Brett Gilio
                   ` (3 subsequent siblings)
  8 siblings, 0 replies; 15+ messages in thread
From: Jesse Gibbons @ 2019-12-10  2:38 UTC (permalink / raw)
  To: 38554

[-- Attachment #1: Type: text/plain, Size: 3 bytes --]




[-- Attachment #2: 0005-gnu-Add-python2-funcparserlib.patch --]
[-- Type: text/x-patch, Size: 839 bytes --]

From 694504baa449b8bd6ff73ca37edcc6dd457fdba1 Mon Sep 17 00:00:00 2001
From: Jesse Gibbons <jgibbons2357+guix@gmail.com>
Date: Mon, 9 Dec 2019 19:17:28 -0700
Subject: [PATCH 5/5] gnu: Add python2-funcparserlib

* gnu/packages/python-xyz.scm (python2-funcparserlib): New variable.
---
 gnu/packages/python-xyz.scm | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index 9c476b0bce..06dd30cbc5 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -16830,3 +16830,7 @@ converters and more, those based on the library itself.")
     (description
      "Recursive descent parsing library based on functional combinators")
     (license license:expat)))
+
+(define-public python2-funcparserlib
+  (package-with-python2 python-funcparserlib))
+
-- 
2.24.0


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

* [bug#38554] [PATCH 0/5]  Update python-hy to 0.17.0
  2019-12-10  2:29 [bug#38554] [PATCH 0/5] Update python-hy to 0.17.0 Jesse Gibbons
                   ` (4 preceding siblings ...)
  2019-12-10  2:38 ` [bug#38554] [PATCH 5/5] Add python2-funcparserlib Jesse Gibbons
@ 2019-12-10  4:25 ` Brett Gilio
  2019-12-10 21:59   ` Jesse Gibbons
  2019-12-21 17:14 ` [bug#38554] [PATCH v2 1/3] gnu: Add python-fastentrypoints Jesse Gibbons
                   ` (2 subsequent siblings)
  8 siblings, 1 reply; 15+ messages in thread
From: Brett Gilio @ 2019-12-10  4:25 UTC (permalink / raw)
  To: Jesse Gibbons; +Cc: 38554

Jesse Gibbons <jgibbons2357@gmail.com> writes:

> The next five patches:
> - add dependencies for python-hy 0.17.0 (patches 1,2)
> - update python-hy from 0.13.0 to 0.17.0 (patch 3)
> - add dependencies for python2-hy 0.17.0 (patches 4,5)
>
>
> This will be the last version of python-hy to support python 2. If it is
> better to drop python2-hy now, simply omit patches 4 and 5, and remove
> python2-hy.

Hy Jesse! (Haha.)

Thank you for your patch series. I am sorry for the delay and the lack
of communication regarding your bug report. I could imagine this is
really frustrating. There are some corrections that I think need to be
made before we go forward, and for the sake of education I would like to
propose some revisions. Once we get these revisions, I think we will be
golden for a push to master! So bare with me.

First off, _USUALLY_ we have the package-for-python2 derivatives in the
same commits as the package indicating the original. Thus, your
python2-* patches (4 & 5) can be squashed into their originating commits
(1 & 2) respectively.

In patch 1: You indicate your copyright email as such:

--8<---------------cut here---------------start------------->8---
;;; Copyright © 2019 Jesse Gibbons <jgibbons2357+guix@gmail.com>
--8<---------------cut here---------------end--------------->8---

Is this correct? Just seems odd to me.

Also in that patch you have too many white space return indications between the
preceeding package, and you have white space return at the EOF. This should be
reduced to one and none, respectively. Next, if you could run `guix
lint` on this package. Your synopsis and description sections are
identical and do not conform to our guidelines. I suspect this was just
the default from running `guix import pypi`.

If you need help with making synopsis and descriptions, here is the
relevant documentation:
https://guix.gnu.org/manual/en/html_node/Synopses-and-Descriptions.html

In patch 2: There should be a white space return before the package in
this case, but was likely cut off from the improper EOF return in patch
1. Otherwise, this package looks okay aside from the issue with the
synopsis and description.

Lastly, all of the commit messages need a period at the end. And the
sub-header message for the patch which bumps Hy to 0.17 needs a period
as well. You could also remove the redundant [inputs] section and simply
make a comma-separated-list of new inputs. Also, if you could add
something like,

--8<---------------cut here---------------start------------->8---
Fixes: <link-to-original-bug-report>.
--8<---------------cut here---------------end--------------->8---

to the patch that updates Hy, that way we can know where this patch
series is predicated from. That would be superb!

Once we get these easy changes done we will push! If you could send your
revisions with something like:

--8<---------------cut here---------------start------------->8---
[PATCH v2 x/3]
--8<---------------cut here---------------end--------------->8---


Again, I apologize for the radio silence.

-- 
Brett M. Gilio
Homepage -- https://scm.pw/
GNU Guix -- https://guix.gnu.org/

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

* [bug#38554] [PATCH 0/5]  Update python-hy to 0.17.0
  2019-12-10  4:25 ` [bug#38554] [PATCH 0/5] Update python-hy to 0.17.0 Brett Gilio
@ 2019-12-10 21:59   ` Jesse Gibbons
  2019-12-12 17:23     ` zimoun
  0 siblings, 1 reply; 15+ messages in thread
From: Jesse Gibbons @ 2019-12-10 21:59 UTC (permalink / raw)
  To: Brett Gilio; +Cc: 38554

On Mon, 2019-12-09 at 22:25 -0600, Brett Gilio wrote:
> Jesse Gibbons <jgibbons2357@gmail.com> writes:
> 
> > The next five patches:
> > - add dependencies for python-hy 0.17.0 (patches 1,2)
> > - update python-hy from 0.13.0 to 0.17.0 (patch 3)
> > - add dependencies for python2-hy 0.17.0 (patches 4,5)
> > 
> > 
> > This will be the last version of python-hy to support python 2. If it is
> > better to drop python2-hy now, simply omit patches 4 and 5, and remove
> > python2-hy.
> 
> Hy Jesse! (Haha.)
> 
> Thank you for your patch series. I am sorry for the delay and the lack
> of communication regarding your bug report. I could imagine this is
> really frustrating. There are some corrections that I think need to be
> made before we go forward, and for the sake of education I would like to
> propose some revisions. Once we get these revisions, I think we will be
> golden for a push to master! So bare with me.
I feel the need to automate producing these patches because there are a lot
of things my poor memory needs to remember. I'll make some commit scripts
and share them later. Or maybe I can write a local git hook to do all those
things for me.
> 
> First off, _USUALLY_ we have the package-for-python2 derivatives in the
> same commits as the package indicating the original. Thus, your
> python2-* patches (4 & 5) can be squashed into their originating commits
> (1 & 2) respectively.
I tried to figure this out, but it has been so long since a python package
has been added, and we anticipate python2 EOL, so I thought I would keep
them separate in case it wasn't worth adding the python2 versions as their
own patches. I'll fix this.
> 
> In patch 1: You indicate your copyright email as such:
> 
> --8<---------------cut here---------------start------------->8---
> ;;; Copyright © 2019 Jesse Gibbons <jgibbons2357+guix@gmail.com>
> --8<---------------cut here---------------end--------------->8---
> 
> Is this correct? Just seems odd to me.

Yes, this is correct. As I understand it, names and email addresses are
added to the copyright section to allow someone to initiate contact with
each of the copyright holders regarding their copyright. The address looks
odd (and doesn't match some of the more common regexps for email addresses)
because it takes advantage of a feature that gives me (theoretically)
infinite email addresses that go to the same inbox. I use it to filter my
email, so I will know (in this case) an email is about a contribution I made
to guix, and my email client can filter it accordingly. I add the same email
address by my name in the copyright region when I send a major patch to
guix, and would rather not change it for this reason.

FMI see https://www.wikihow.com/Use-Plus-Addressing-in-Gmail
> 
> Also in that patch you have too many white space return indications
> between the preceeding package, and you have white space return at the
> EOF. This should be reduced to one and none, respectively.
Ok. So do I understand correctly patch 1 should have a single white space
after the package, but not patch 2?

> Next, if you could run `guix lint` on this package. Your synopsis and
> description sections are identical and do not conform to our guidelines. I
> suspect this was just the default from running `guix import pypi`.
> 
> If you need help with making synopsis and descriptions, here is the
> relevant documentation:
> https://guix.gnu.org/manual/en/html_node/Synopses-and-Descriptions.html
I forgot to change the synopsis and descriptions. I'll make sure I do that
before I send the fixed patches.
> 
> In patch 2: There should be a white space return before the package in
> this case, but was likely cut off from the improper EOF return in patch
> 1. Otherwise, this package looks okay aside from the issue with the
> synopsis and description.
> 
> Lastly, all of the commit messages need a period at the end. And the
> sub-header message for the patch which bumps Hy to 0.17 needs a period
> as well. You could also remove the redundant [inputs] section and simply
> make a comma-separated-list of new inputs. Also, if you could add
> something like,
> 
> --8<---------------cut here---------------start------------->8---
> Fixes: <link-to-original-bug-report>.
> --8<---------------cut here---------------end--------------->8---
> 
> to the patch that updates Hy, that way we can know where this patch
> series is predicated from. That would be superb!
Ok. I will make sure that happens.
> 
> Once we get these easy changes done we will push! If you could send your
> revisions with something like:
> 
> --8<---------------cut here---------------start------------->8---
> [PATCH v2 x/3]
> --8<---------------cut here---------------end--------------->8---
> 
In response to this email, or simply to 38554@debbugs.gnu.org? I'm guessing
either will work, but you are more likely to see a response to this email.
> 
> Again, I apologize for the radio silence.
> 
Not to worry. I had (and still have) other work to do, and this was a low-
priority task. I might not be able to send the fixed patches until possibly
the 18th or 19th. I'll send the corrected patches, and possibly contribute
other pending package definitions and/or bug reports as well.

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

* [bug#38554] [PATCH 4/5] Add python2-fastentrypoints
  2019-12-10  2:37 ` [bug#38554] [PATCH 4/5] Add python2-fastentrypoints Jesse Gibbons
@ 2019-12-12 17:18   ` zimoun
  2019-12-16  0:37     ` Jesse Gibbons
  0 siblings, 1 reply; 15+ messages in thread
From: zimoun @ 2019-12-12 17:18 UTC (permalink / raw)
  To: Jesse Gibbons; +Cc: 38554

Hi Jesse,

Thank you for your patches.

As you have maybe seen, I have file a bug [1] to track the Python
packages and the end-of-file of Python 2. Here you add a new Python 2
entry. My question is: do we absolutely need it?

I have seen your bug report about the broken Hy REPL and this series
fixes it. My question is just: can we continue to add
`package-with-python2` packages?


All the best,
simon

[1] https://debbugs.gnu.org/cgi/bugreport.cgi?bug=38420

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

* [bug#38554] [PATCH 0/5] Update python-hy to 0.17.0
  2019-12-10 21:59   ` Jesse Gibbons
@ 2019-12-12 17:23     ` zimoun
  0 siblings, 0 replies; 15+ messages in thread
From: zimoun @ 2019-12-12 17:23 UTC (permalink / raw)
  To: Jesse Gibbons; +Cc: Brett Gilio, 38554

Hi,

On Tue, 10 Dec 2019 at 23:01, Jesse Gibbons <jgibbons2357@gmail.com> wrote:

> > Once we get these easy changes done we will push! If you could send your
> > revisions with something like:
> >
> > --8<---------------cut here---------------start------------->8---
> > [PATCH v2 x/3]
> > --8<---------------cut here---------------end--------------->8---
> >
> In response to this email, or simply to 38554@debbugs.gnu.org? I'm guessing
> either will work, but you are more likely to see a response to this email.

Yes to 38554@debbugs.gnu.org using the Git option --reroll-count=2 :-)

Thank you for your patches. Hy is really fun. :-)

All the best,
simon

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

* [bug#38554] [PATCH 4/5] Add python2-fastentrypoints
  2019-12-12 17:18   ` zimoun
@ 2019-12-16  0:37     ` Jesse Gibbons
  0 siblings, 0 replies; 15+ messages in thread
From: Jesse Gibbons @ 2019-12-16  0:37 UTC (permalink / raw)
  To: zimoun; +Cc: 38554

On Thu, 2019-12-12 at 18:18 +0100, zimoun wrote:
> Hi Jesse,
> 
> Thank you for your patches.
> 
> As you have maybe seen, I have file a bug [1] to track the Python
> packages and the end-of-file of Python 2. Here you add a new Python 2
> entry. My question is: do we absolutely need it?
> 
> I have seen your bug report about the broken Hy REPL and this series
> fixes it. My question is just: can we continue to add
> `package-with-python2` packages?
> 
> 
> All the best,
> simon
> 
> [1] https://debbugs.gnu.org/cgi/bugreport.cgi?bug=38420
This concern is why I added the python2 packages in separate patches
contrary to tradition. I wanted to know what others thought of it. We don't
necessarily need to add the python2 packages, especially when the repository
says this will be the last version of hy to support python2 and those
packages are only added to support hy in python2. I recommend you start a
discussion with Brett, who reviewed the patches and told me to combine patch
1 with 4 and 2 with 5. 
I have too much going on to create and send updated patches before Tuesday,
so I guess now is the time for such discussion.

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

* [bug#38554] [PATCH v2 1/3] gnu: Add python-fastentrypoints.
  2019-12-10  2:29 [bug#38554] [PATCH 0/5] Update python-hy to 0.17.0 Jesse Gibbons
                   ` (5 preceding siblings ...)
  2019-12-10  4:25 ` [bug#38554] [PATCH 0/5] Update python-hy to 0.17.0 Brett Gilio
@ 2019-12-21 17:14 ` Jesse Gibbons
  2019-12-21 17:14 ` [bug#38554] [PATCH v2 2/3] gnu: Add python-funcparserlib Jesse Gibbons
  2019-12-21 17:14 ` [bug#38554] [PATCH v2 3/3] gnu: python-hy: Update to 0.17.0 Jesse Gibbons
  8 siblings, 0 replies; 15+ messages in thread
From: Jesse Gibbons @ 2019-12-21 17:14 UTC (permalink / raw)
  To: 38554

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

diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index 520de4607d..9243272dc9 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -68,6 +68,7 @@
 ;;; Copyright © 2019 Wiktor Żelazny <wzelazny@vurv.cz>
 ;;; Copyright © 2019 Tanguy Le Carrour <tanguy@bioneland.org>
 ;;; Copyright © 2019 Mădălin Ionel Patrașcu <
madalinionel.patrascu@mdc-berlin.de>
+;;; Copyright © 2019 Jesse Gibbons <jgibbons2357+guix@gmail.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -16972,3 +16973,28 @@ as @command{which} and @command{find}.  These
commands allow automated
 discovery of what has been installed on an operating system, and where the
 essential tools are located.")
     (license license:bsd-3)))
+
+(define-public python-fastentrypoints
+  (package
+    (name "python-fastentrypoints")
+    (version "0.12")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (pypi-uri "fastentrypoints" version))
+       (sha256
+        (base32
+         "02s1j8i2dzbpbwgq2a3fiqwm3cnmhii2qzc0k42l0rdxd4a4ya7z"))))
+    (build-system python-build-system)
+    (home-page
+     "https://github.com/ninjaaron/fast-entry_points")
+    (synopsis
+     "Makes entry_points specified in setup.py load more quickly")
+    (description
+     "Using entry_points in your setup.py makes scripts that start really
slowly
+because it imports pkg_resources.  This package allows such setup scripts
to
+load entry points more quickly.")
+    (license license:bsd-3)))
+
+(define-public python2-fastentrypoints
+  (package-with-python2 python-fastentrypoints))

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

* [bug#38554] [PATCH v2 2/3] gnu: Add python-funcparserlib.
  2019-12-10  2:29 [bug#38554] [PATCH 0/5] Update python-hy to 0.17.0 Jesse Gibbons
                   ` (6 preceding siblings ...)
  2019-12-21 17:14 ` [bug#38554] [PATCH v2 1/3] gnu: Add python-fastentrypoints Jesse Gibbons
@ 2019-12-21 17:14 ` Jesse Gibbons
  2019-12-21 17:14 ` [bug#38554] [PATCH v2 3/3] gnu: python-hy: Update to 0.17.0 Jesse Gibbons
  8 siblings, 0 replies; 15+ messages in thread
From: Jesse Gibbons @ 2019-12-21 17:14 UTC (permalink / raw)
  To: 38554

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

diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index 9243272dc9..906bd9c8d2 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -16998,3 +16998,52 @@ load entry points more quickly.")
 
 (define-public python2-fastentrypoints
   (package-with-python2 python-fastentrypoints))
+
+(define-public python-funcparserlib
+  (package
+    (name "python-funcparserlib")
+    (version "0.3.6")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (pypi-uri "funcparserlib" version))
+       (sha256
+        (base32
+         "07f9cgjr3h4j2m67fhwapn8fja87vazl58zsj4yppf9y3an2x6dp"))))
+    (native-inputs
+     `(("python-tox" ,python-tox)))
+    (arguments
+     `(#:phases
+       (modify-phases %standard-phases
+         (replace 'check
+           (lambda _
+             (invoke "tox"))))))
+    (build-system python-build-system)
+    (home-page
+     "https://github.com/vlasovskikh/funcparserlib")
+    (synopsis
+     "Recursive descent parsing library based on functional combinators")
+    (description
+     "This package is a recurisve descent parsing library for Python based
on
+functional combinators.  Parser combinators are just higher-order functions
that
+take parsers as their arguments and return them as result values.  Parser
+combinators are:
+
+@itemize @bullet
+@item First-class values
+@item Extremely composable
+@item Tend to make the code quite compact
+@item Resemble the readable notation of xBNF grammars
+@end itemize
+
+Parsers made with funcparserlib are pure-Python LL(*) parsers.  It means
that
+it's very easy to write them without thinking about look-aheads and all
that
+hardcore parsing stuff.  But the recursive descent parsing is a rather slow
+method compared to LL(k) or LR(k) algorithms.
+
+So the primary domain for funcparserlib is parsing little languages or
external
+DSLs (domain specific languages).")
+    (license license:expat)))
+
+(define-public python2-funcparserlib
+  (package-with-python2 python-funcparserlib))

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

* [bug#38554] [PATCH v2 3/3] gnu: python-hy: Update to 0.17.0.
  2019-12-10  2:29 [bug#38554] [PATCH 0/5] Update python-hy to 0.17.0 Jesse Gibbons
                   ` (7 preceding siblings ...)
  2019-12-21 17:14 ` [bug#38554] [PATCH v2 2/3] gnu: Add python-funcparserlib Jesse Gibbons
@ 2019-12-21 17:14 ` Jesse Gibbons
  2019-12-25  7:27   ` bug#38554: " Brett Gilio
  8 siblings, 1 reply; 15+ messages in thread
From: Jesse Gibbons @ 2019-12-21 17:14 UTC (permalink / raw)
  To: 38554

* gnu/packages/python-xyz.scm (python-hy): Update to 0.17.0.
[inputs]: Add python-fastentrypoints, python-funcparserlib.
Fixes: https://lists.gnu.org/archive/html/bug-guix/2019-12/msg00034.html.
---
 gnu/packages/python-xyz.scm | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index 906bd9c8d2..c5dc022475 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -8514,20 +8514,20 @@ with a new public API, and RPython support.")
 (define-public python-hy
   (package
     (name "python-hy")
-    (version "0.13.0")
+    (version "0.17.0")
     (source (origin
               (method url-fetch)
               (uri (pypi-uri "hy" version))
               (sha256
                (base32
-                "19sfymaksx9jhksfnb15ahid46mzrhdfzz6yy2craz2qnzvpmky8"))))
+                "1gdbqsirsdxj320wnp7my5awzs1kfs6m4fqmkzbd1zd47qzj0zfi"))))
     (build-system python-build-system)
     (arguments
      '(#:phases
        (modify-phases %standard-phases
-	 (add-before 'install 'set-HOME
-	   (lambda _
-	     (setenv "HOME" "/tmp")))
+         (add-before 'install 'set-HOME
+           (lambda _
+             (setenv "HOME" "/tmp")))
          (replace 'check
            (lambda _
              ;; Tests require write access to HOME.
@@ -8539,8 +8539,10 @@ with a new public API, and RPython support.")
     (propagated-inputs
      `(("python-astor" ,python-astor)
        ("python-clint" ,python-clint)
-       ("python-rply" ,python-rply)))
-    (home-page "http://hylang.org/")
+       ("python-rply" ,python-rply)
+       ("python-fastentrypoints" ,python-fastentrypoints)
+       ("python-funcparserlib" ,python-funcparserlib)))
+    (home-page "http://docs.hylang.org/")
     (synopsis "Lisp frontend to Python")
     (description
      "Hy is a dialect of Lisp that's embedded in Python.  Since Hy
transforms

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

* bug#38554: [PATCH v2 3/3] gnu: python-hy: Update to 0.17.0.
  2019-12-21 17:14 ` [bug#38554] [PATCH v2 3/3] gnu: python-hy: Update to 0.17.0 Jesse Gibbons
@ 2019-12-25  7:27   ` Brett Gilio
  0 siblings, 0 replies; 15+ messages in thread
From: Brett Gilio @ 2019-12-25  7:27 UTC (permalink / raw)
  To: Jesse Gibbons; +Cc: 38554


Hy Jesse! (ha).

Thank you for taking the time to do the revisions I suggested. I
actually could not apply your patches cleanly (not your fault). So I
went through and manually added and modified the patches as
necessary. You are listed as the author on all of the commits.

I tested the resultant Hy package, and it looks and works fine! I do
think eventually we should maybe consider moving Hy to its own file
though? With a name that is just "hy" and not "python-hy". Maybe a
discussion for a different day.

6b520177734e1c144fd6394f801919d0b54fa23e
c5eb8cf844148fb99934c3fedc3b4a0da55f9d8e
c215051dcf30b11c755aea4e2527b4d3d91c34bb

Thanks! Closing.

-- 
Brett M. Gilio
GNU Guix, Contributor | GNU Project, Webmaster
[DFC0 C7F7 9EE6 0CA7 AE55 5E19 6722 43C4 A03F 0EEE]
<brettg@gnu.org> <brettg@posteo.net>

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

end of thread, other threads:[~2019-12-25  7:28 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-12-10  2:29 [bug#38554] [PATCH 0/5] Update python-hy to 0.17.0 Jesse Gibbons
2019-12-10  2:35 ` [bug#38554] [PATCH 1/5] Add python-fastentrypoints Jesse Gibbons
2019-12-10  2:36 ` [bug#38554] [PATCH 2/5] Add python-funcparserlib Jesse Gibbons
2019-12-10  2:37 ` [bug#38554] [PATCH 3/5] Update python-hy to 0.17.0 Jesse Gibbons
2019-12-10  2:37 ` [bug#38554] [PATCH 4/5] Add python2-fastentrypoints Jesse Gibbons
2019-12-12 17:18   ` zimoun
2019-12-16  0:37     ` Jesse Gibbons
2019-12-10  2:38 ` [bug#38554] [PATCH 5/5] Add python2-funcparserlib Jesse Gibbons
2019-12-10  4:25 ` [bug#38554] [PATCH 0/5] Update python-hy to 0.17.0 Brett Gilio
2019-12-10 21:59   ` Jesse Gibbons
2019-12-12 17:23     ` zimoun
2019-12-21 17:14 ` [bug#38554] [PATCH v2 1/3] gnu: Add python-fastentrypoints Jesse Gibbons
2019-12-21 17:14 ` [bug#38554] [PATCH v2 2/3] gnu: Add python-funcparserlib Jesse Gibbons
2019-12-21 17:14 ` [bug#38554] [PATCH v2 3/3] gnu: python-hy: Update to 0.17.0 Jesse Gibbons
2019-12-25  7:27   ` bug#38554: " Brett Gilio

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