GNU bug report logs - #57744
[PATCH 0/3] Add shirah and dependencies

Previous Next

Package: guix-patches;

Reported by: M <matf <at> disr.it>

Date: Mon, 12 Sep 2022 00:10:02 UTC

Severity: normal

Tags: patch

Done: Christopher Baines <mail <at> cbaines.net>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 57744 in the body.
You can then email your comments to 57744 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to guix-patches <at> gnu.org:
bug#57744; Package guix-patches. (Mon, 12 Sep 2022 00:10:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to M <matf <at> disr.it>:
New bug report received and forwarded. Copy sent to guix-patches <at> gnu.org. (Mon, 12 Sep 2022 00:10:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: M <matf <at> disr.it>
To: guix-patches <at> gnu.org
Cc: M <matf <at> disr.it>
Subject: [PATCH 0/3] Add shirah and dependencies
Date: Mon, 12 Sep 2022 02:09:15 +0200
shirah is a terminal-based ebook reader, which alone I'd see as a nice addition to Guix packages considering how light, flexible and customizable (themes, opacity, font options, keybindings) terminal emulators are, but it also comes with an optional RSVP mode for speed reading, which can be pretty convenient for a terminal reader since terminal windows can be resized to very small geometry without breaking. shirah also supports basic ebook-reading features, like resuming a read, or browsing from the TOC.

M (3):
  gnu: Add python-ebooklib.
  gnu: Add python-syllables.
  gnu: Add shirah.

 gnu/packages/ebook.scm      | 40 +++++++++++++++++++++++++++++++++++++
 gnu/packages/python-xyz.scm | 20 +++++++++++++++++++
 2 files changed, 60 insertions(+)

-- 
2.37.2





Information forwarded to guix-patches <at> gnu.org:
bug#57744; Package guix-patches. (Mon, 12 Sep 2022 00:11:02 GMT) Full text and rfc822 format available.

Message #8 received at 57744 <at> debbugs.gnu.org (full text, mbox):

From: M <matf <at> disr.it>
To: 57744 <at> debbugs.gnu.org
Cc: M <matf <at> disr.it>
Subject: [PATCH 1/3] gnu: Add python-ebooklib.
Date: Mon, 12 Sep 2022 02:10:26 +0200
* gnu/packages/ebook.scm (python-ebooklib): New variable.
---
 gnu/packages/ebook.scm | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

diff --git a/gnu/packages/ebook.scm b/gnu/packages/ebook.scm
index c3a27cb113..be41318095 100644
--- a/gnu/packages/ebook.scm
+++ b/gnu/packages/ebook.scm
@@ -10,6 +10,7 @@
 ;;; Copyright © 2020 Zheng Junjie <873216071 <at> qq.com>
 ;;; Copyright © 2021 la snesne <lasnesne <at> lagunposprasihopre.org>
 ;;; Copyright © 2021 Petr Hodina <phodina <at> protonmail.com>
+;;; Copyright © 2021 Mathieu Laparie <mlaparie <at> disr.it>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -678,3 +679,21 @@ (define-public libmobi
 @item handling encrypted documents
 @end itemize\n")
     (license license:lgpl3+)))
+
+(define-public python-ebooklib
+  (package
+    (name "python-ebooklib")
+    (version "0.17.1")
+    (source (origin
+              (method url-fetch)
+              (uri (pypi-uri "EbookLib" version))
+              (sha256
+               (base32
+                "1w972g0kmh9cdxf3kjr7v4k99wvv4lxv3rxkip39c08550nf48zy"))))
+    (build-system python-build-system)
+    (propagated-inputs (list python-lxml python-six))
+    (home-page "https://github.com/aerkalov/ebooklib")
+    (synopsis "Ebook library which can handle EPUB2/EPUB3 and Kindle format")
+    (description "Ebook library which can handle EPUB2/EPUB3 and Kindle
+format.")
+    (license license:agpl3)))
-- 
2.37.2





Information forwarded to guix-patches <at> gnu.org:
bug#57744; Package guix-patches. (Mon, 12 Sep 2022 00:11:02 GMT) Full text and rfc822 format available.

Message #11 received at 57744 <at> debbugs.gnu.org (full text, mbox):

From: M <matf <at> disr.it>
To: 57744 <at> debbugs.gnu.org
Cc: M <matf <at> disr.it>
Subject: [PATCH 2/3] gnu: Add python-syllables.
Date: Mon, 12 Sep 2022 02:10:27 +0200
* gnu/packages/python-xyz.scm (python-syllables): New variable.
---
 gnu/packages/python-xyz.scm | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)

diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index 054bd3ed29..9d358a4926 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -128,6 +128,7 @@
 ;;; Copyright © 2022 Marek Felšöci <marek <at> felsoci.sk>
 ;;; Copyright © 2022 Hilton Chain <hako <at> ultrarare.space>
 ;;; Copyright © 2022 Tomasz Jeneralczyk <tj <at> schwi.pl>
+;;; Copyright © 2022 Mathieu Laparie <mlaparie <at> disr.it>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -10032,6 +10033,25 @@ (define-public python-yte
      "This package provides a YAML template engine with Python expressions.")
     (license license:expat)))
 
+(define-public python-syllables
+  (package
+    (name "python-syllables")
+    (version "1.0.3")
+    (source (origin
+              (method url-fetch)
+              (uri (pypi-uri "syllables" version))
+              (sha256
+               (base32
+                "0wkl6h0rg6fbsxfp0a8fnibf3l4l6lbh6z12cvcilgb6qhxzpmv3"))))
+    (build-system python-build-system)
+    (home-page "https://github.com/prosegrinder/python-syllables")
+    (synopsis
+     "A Python package for estimating the number of syllables in a word.")
+    (description
+     "This package provides a Python package for estimating the number of
+syllables in a word.")
+    (license license:gpl3+)))
+
 (define-public python-sympy
   (package
     (name "python-sympy")
-- 
2.37.2





Information forwarded to guix-patches <at> gnu.org:
bug#57744; Package guix-patches. (Mon, 12 Sep 2022 00:11:02 GMT) Full text and rfc822 format available.

Message #14 received at 57744 <at> debbugs.gnu.org (full text, mbox):

From: M <matf <at> disr.it>
To: 57744 <at> debbugs.gnu.org
Cc: M <matf <at> disr.it>
Subject: [PATCH 3/3] gnu: Add shirah.
Date: Mon, 12 Sep 2022 02:10:28 +0200
* gnu/packages/ebook.scm (shirah): New variable.
---
 gnu/packages/ebook.scm | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/gnu/packages/ebook.scm b/gnu/packages/ebook.scm
index be41318095..a5a19cb9c0 100644
--- a/gnu/packages/ebook.scm
+++ b/gnu/packages/ebook.scm
@@ -697,3 +697,24 @@ (define-public python-ebooklib
     (description "Ebook library which can handle EPUB2/EPUB3 and Kindle
 format.")
     (license license:agpl3)))
+
+(define-public shirah
+  (package
+    (name "shirah")
+    (version "1.0.0")
+    (source (origin
+              (method url-fetch)
+              (uri (pypi-uri "shirah_reader" version))
+              (sha256
+               (base32
+                "0j15v435lz68c1mj5clfx5dmfyjc6jvvz2q8hqvv799mb2faj42y"))))
+    (build-system python-build-system)
+    (propagated-inputs (list python-beautifulsoup4 python-ebooklib
+                             python-syllables python-termcolor))
+    (home-page "https://github.com/hallicopter/shirah-reader")
+    (synopsis "A curses based terminal ebook reader with optional RSVP mode")
+    (description "@command{shirah} is a curses based terminal ebook reader that
+can display ebooks in the usual way or with Rapid Serial Visual Presentation, a
+method to enable speedreading by showing the text word by word at configurable
+speeds.")
+  (license license:expat)))
-- 
2.37.2





Information forwarded to guix-patches <at> gnu.org:
bug#57744; Package guix-patches. (Wed, 14 Sep 2022 08:12:01 GMT) Full text and rfc822 format available.

Message #17 received at 57744 <at> debbugs.gnu.org (full text, mbox):

From: Christopher Baines <mail <at> cbaines.net>
To: M <matf <at> disr.it>
Cc: 57744 <at> debbugs.gnu.org
Subject: Re: [bug#57744] [PATCH 0/3] Add shirah and dependencies
Date: Wed, 14 Sep 2022 09:09:33 +0100
[Message part 1 (text/plain, inline)]
M <matf <at> disr.it> writes:

> shirah is a terminal-based ebook reader, which alone I'd see as a nice
> addition to Guix packages considering how light, flexible and
> customizable (themes, opacity, font options, keybindings) terminal
> emulators are, but it also comes with an optional RSVP mode for speed
> reading, which can be pretty convenient for a terminal reader since
> terminal windows can be resized to very small geometry without
> breaking. shirah also supports basic ebook-reading features, like
> resuming a read, or browsing from the TOC.
>
> M (3):
>   gnu: Add python-ebooklib.
>   gnu: Add python-syllables.
>   gnu: Add shirah.
>
>  gnu/packages/ebook.scm      | 40 +++++++++++++++++++++++++++++++++++++
>  gnu/packages/python-xyz.scm | 20 +++++++++++++++++++
>  2 files changed, 60 insertions(+)


Hey,

These generally look good to me, but I think there are a few things to
fix before merging.

The license information looks off for all packages, can you take a look?

Also, the linter makes some comments about some package synopsis.

Thanks,

Chris
[signature.asc (application/pgp-signature, inline)]

Information forwarded to guix-patches <at> gnu.org:
bug#57744; Package guix-patches. (Thu, 15 Sep 2022 19:58:01 GMT) Full text and rfc822 format available.

Message #20 received at 57744 <at> debbugs.gnu.org (full text, mbox):

From: Mathieu <matf <at> disr.it>
To: Christopher Baines <mail <at> cbaines.net>
Cc: 57744 <at> debbugs.gnu.org
Subject: Re: [bug#57744] [PATCH 0/3] Add shirah and dependencies
Date: Thu, 15 Sep 2022 21:57:42 +0200
Thanks for the review! I fixed the synopsis and licenses for shirah and python-syllables, but python-ebooklib's agpl3 is correct, is it not?

On 2022-09-14 10:09 Christopher Baines <mail <at> cbaines.net> wrote:

>  
> M <matf <at> disr.it> writes:
>  
>> shirah is a terminal-based ebook reader, which alone I'd see as a
>> nice
>> addition to Guix packages considering how light, flexible and
>> customizable (themes, opacity, font options, keybindings) terminal
>> emulators are, but it also comes with an optional RSVP mode for
>> speed
>> reading, which can be pretty convenient for a terminal reader since
>> terminal windows can be resized to very small geometry without
>> breaking. shirah also supports basic ebook-reading features, like
>> resuming a read, or browsing from the TOC.
>>  
>> M (3):
>> gnu: Add python-ebooklib.
>> gnu: Add python-syllables.
>> gnu: Add shirah.
>>  
>> gnu/packages/ebook.scm      | 40
>> +++++++++++++++++++++++++++++++++++++
>> gnu/packages/python-xyz.scm | 20 +++++++++++++++++++
>> 2 files changed, 60 insertions(+)
>  
>  
> Hey,
>  
> These generally look good to me, but I think there are a few things
> to
> fix before merging.
>  
> The license information looks off for all packages, can you take a
> look?
>  
> Also, the linter makes some comments about some package synopsis.
>  
> Thanks,
>  
> Chris




Information forwarded to guix-patches <at> gnu.org:
bug#57744; Package guix-patches. (Thu, 15 Sep 2022 19:59:02 GMT) Full text and rfc822 format available.

Message #23 received at 57744 <at> debbugs.gnu.org (full text, mbox):

From: M <matf <at> disr.it>
To: 57744 <at> debbugs.gnu.org
Cc: M <matf <at> disr.it>
Subject: [PATCH v2 1/3] gnu: Add shirah.
Date: Thu, 15 Sep 2022 21:58:15 +0200
* gnu/packages/ebook.scm (shirah): New variable.
---
 gnu/packages/ebook.scm | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/gnu/packages/ebook.scm b/gnu/packages/ebook.scm
index be41318095..a5a19cb9c0 100644
--- a/gnu/packages/ebook.scm
+++ b/gnu/packages/ebook.scm
@@ -697,3 +697,24 @@ (define-public python-ebooklib
     (description "Ebook library which can handle EPUB2/EPUB3 and Kindle
 format.")
     (license license:agpl3)))
+
+(define-public shirah
+  (package
+    (name "shirah")
+    (version "1.0.0")
+    (source (origin
+              (method url-fetch)
+              (uri (pypi-uri "shirah_reader" version))
+              (sha256
+               (base32
+                "0j15v435lz68c1mj5clfx5dmfyjc6jvvz2q8hqvv799mb2faj42y"))))
+    (build-system python-build-system)
+    (propagated-inputs (list python-beautifulsoup4 python-ebooklib
+                             python-syllables python-termcolor))
+    (home-page "https://github.com/hallicopter/shirah-reader")
+    (synopsis "A curses based terminal ebook reader with optional RSVP mode")
+    (description "@command{shirah} is a curses based terminal ebook reader that
+can display ebooks in the usual way or with Rapid Serial Visual Presentation, a
+method to enable speedreading by showing the text word by word at configurable
+speeds.")
+  (license license:expat)))
-- 
2.37.2





Information forwarded to guix-patches <at> gnu.org:
bug#57744; Package guix-patches. (Thu, 15 Sep 2022 19:59:02 GMT) Full text and rfc822 format available.

Message #26 received at 57744 <at> debbugs.gnu.org (full text, mbox):

From: M <matf <at> disr.it>
To: 57744 <at> debbugs.gnu.org
Cc: M <matf <at> disr.it>
Subject: [PATCH v2 2/3] gnu: Add shirah.
Date: Thu, 15 Sep 2022 21:58:16 +0200
* gnu/packages/ebook.scm (shirah): New variable.
---
 gnu/packages/ebook.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/ebook.scm b/gnu/packages/ebook.scm
index a5a19cb9c0..2ade4f4d7e 100644
--- a/gnu/packages/ebook.scm
+++ b/gnu/packages/ebook.scm
@@ -712,9 +712,9 @@ (define-public shirah
     (propagated-inputs (list python-beautifulsoup4 python-ebooklib
                              python-syllables python-termcolor))
     (home-page "https://github.com/hallicopter/shirah-reader")
-    (synopsis "A curses based terminal ebook reader with optional RSVP mode")
+    (synopsis "Terminal ebook reader with an optional RSVP mode")
     (description "@command{shirah} is a curses based terminal ebook reader that
 can display ebooks in the usual way or with Rapid Serial Visual Presentation, a
 method to enable speedreading by showing the text word by word at configurable
 speeds.")
-  (license license:expat)))
+  (license license:gpl2)))
-- 
2.37.2





Information forwarded to guix-patches <at> gnu.org:
bug#57744; Package guix-patches. (Thu, 15 Sep 2022 20:00:02 GMT) Full text and rfc822 format available.

Message #29 received at 57744 <at> debbugs.gnu.org (full text, mbox):

From: M <matf <at> disr.it>
To: 57744 <at> debbugs.gnu.org
Cc: M <matf <at> disr.it>
Subject: [PATCH v2 3/3] gnu: Add python-syllables.
Date: Thu, 15 Sep 2022 21:58:17 +0200
* gnu/packages/python-xyz.scm (python-syllables): New variable.
---
 gnu/packages/python-xyz.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index 9d358a4926..b64f2441bf 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -10050,7 +10050,7 @@ (define-public python-syllables
     (description
      "This package provides a Python package for estimating the number of
 syllables in a word.")
-    (license license:gpl3+)))
+    (license license:gpl3)))
 
 (define-public python-sympy
   (package
-- 
2.37.2





Information forwarded to guix-patches <at> gnu.org:
bug#57744; Package guix-patches. (Thu, 15 Sep 2022 20:00:03 GMT) Full text and rfc822 format available.

Message #32 received at 57744 <at> debbugs.gnu.org (full text, mbox):

From: Maxime Devos <maximedevos <at> telenet.be>
To: Mathieu <matf <at> disr.it>, Christopher Baines <mail <at> cbaines.net>
Cc: 57744 <at> debbugs.gnu.org
Subject: Re: [bug#57744] [PATCH 0/3] Add shirah and dependencies
Date: Thu, 15 Sep 2022 21:59:55 +0200
[Message part 1 (text/plain, inline)]

On 15-09-2022 21:57, Mathieu wrote:
> Thanks for the review! I fixed the synopsis and licenses for shirah and python-syllables, but python-ebooklib's agpl3 is correct, is it not?

Looking at 
<https://github.com/aerkalov/ebooklib/blob/master/ebooklib/epub.py>, it 
appears to be agpl3+, not agpl3.

Greetings,
Maxime.
[OpenPGP_0x49E3EE22191725EE.asc (application/pgp-keys, attachment)]
[OpenPGP_signature (application/pgp-signature, attachment)]

Information forwarded to guix-patches <at> gnu.org:
bug#57744; Package guix-patches. (Thu, 15 Sep 2022 20:02:01 GMT) Full text and rfc822 format available.

Message #35 received at 57744 <at> debbugs.gnu.org (full text, mbox):

From: Maxime Devos <maximedevos <at> telenet.be>
To: M <matf <at> disr.it>, 57744 <at> debbugs.gnu.org
Subject: Re: [bug#57744] [PATCH v2 2/3] gnu: Add shirah.
Date: Thu, 15 Sep 2022 22:01:00 +0200
[Message part 1 (text/plain, inline)]

On 15-09-2022 21:58, M wrote:
> +  (license license:gpl2)))

Going by 
<https://github.com/Hallicopter/shirah-reader/blob/main/shirah_reader/__main__.py>, 
it appears t be gpl2+, not gpl2.
[OpenPGP_0x49E3EE22191725EE.asc (application/pgp-keys, attachment)]
[OpenPGP_signature (application/pgp-signature, attachment)]

Information forwarded to guix-patches <at> gnu.org:
bug#57744; Package guix-patches. (Thu, 15 Sep 2022 20:04:02 GMT) Full text and rfc822 format available.

Message #38 received at 57744 <at> debbugs.gnu.org (full text, mbox):

From: Mathieu <matf <at> disr.it>
To: Maxime Devos <maximedevos <at> telenet.be>
Cc: 57744 <at> debbugs.gnu.org
Subject: Re: [bug#57744] [PATCH v2 2/3] gnu: Add shirah.
Date: Thu, 15 Sep 2022 22:03:17 +0200
Should I not use what they have in the LICENSE at the root of their repository instead?

I messed up my patchset v2, still learning the git send-email workflow, sorry for the clutter messages.

On 2022-09-15 22:01 Maxime Devos <maximedevos <at> telenet.be> wrote:

>  
>  
> On 15-09-2022 21:58, M wrote:
>> +  (license license:gpl2)))
>  
> Going by
> <https://github.com/Hallicopter/shirah-reader/blob/main/shirah_reader/
> __main__.py>, it appears t be gpl2+, not gpl2.




Information forwarded to guix-patches <at> gnu.org:
bug#57744; Package guix-patches. (Thu, 15 Sep 2022 20:19:02 GMT) Full text and rfc822 format available.

Message #41 received at 57744 <at> debbugs.gnu.org (full text, mbox):

From: Maxime Devos <maximedevos <at> telenet.be>
To: Mathieu <matf <at> disr.it>
Cc: 57744 <at> debbugs.gnu.org
Subject: Re: [bug#57744] [PATCH v2 2/3] gnu: Add shirah.
Date: Thu, 15 Sep 2022 22:17:07 +0200
[Message part 1 (text/plain, inline)]

On 15-09-2022 22:03, Mathieu wrote:
> Should I not use what they have in the LICENSE at the root of their repository instead?

shirah_reader does not have a LICENSE file, AFAICS.

Even if it did, usually the LICENSE file only contains the license text, 
and not extras like 'which version range'.

Even if there was a LICENSE file that states 'this version only', then 
there is still another file that states 'this version or later', then 
how would we know which one is correct?  Why would LICENSE have 
precedence?  Or why would the source file have precedence?

When such situations happen, we cannot resolve them, only the author can 
clarify matters.

Even if there is only a LICENSE file, then usually it does not specify 
‘this version only’ or ‘this version or later’.  In case of the GPL, we 
then have the following license condition:

‘14. Revised Versions of this License

Each version is given a distinguishing version number.  If the
Program specifies that a certain numbered version of the GNU General
Public License "or any later version" applies to it, you have the
option of following the terms and conditions either of that numbered
version or of any later version published by the Free Software
Foundation.  _If the Program does not specify a version number of the
GNU General Public License, you may choose any version ever published
by the Free Software Foundation._’

I don't consider adding a LICENSE file as ‘specifying a version number’, 
so I'd say that in such cases it's gpl1+ (as the license text needs to 
be distributed along the source code anyway (at least in v2 and v3)). 
(However, I suppose it could be argued that in such cases it's 
implicitly specified that it's ‘version-of-the-LICENSE-file-or-later’. 
Also not a lawyer.  And in such situations, maybe it would be best to 
just ask the author what they meant.)

Greetings,
Maxime.
[OpenPGP_0x49E3EE22191725EE.asc (application/pgp-keys, attachment)]
[OpenPGP_signature (application/pgp-signature, attachment)]

Information forwarded to guix-patches <at> gnu.org:
bug#57744; Package guix-patches. (Thu, 15 Sep 2022 20:39:01 GMT) Full text and rfc822 format available.

Message #44 received at 57744 <at> debbugs.gnu.org (full text, mbox):

From: M <matf <at> disr.it>
To: 57744 <at> debbugs.gnu.org
Cc: M <matf <at> disr.it>
Subject: [PATCH v3 2/3] gnu: Add python-syllables.
Date: Thu, 15 Sep 2022 22:38:01 +0200
* gnu/packages/python-xyz.scm (python-syllables): New variable.
---
 gnu/packages/python-xyz.scm | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index 054bd3ed29..1b68209c67 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -128,6 +128,7 @@
 ;;; Copyright © 2022 Marek Felšöci <marek <at> felsoci.sk>
 ;;; Copyright © 2022 Hilton Chain <hako <at> ultrarare.space>
 ;;; Copyright © 2022 Tomasz Jeneralczyk <tj <at> schwi.pl>
+;;; Copyright © 2022 Mathieu Laparie <mlaparie <at> disr.it>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -10032,6 +10033,24 @@ (define-public python-yte
      "This package provides a YAML template engine with Python expressions.")
     (license license:expat)))
 
+(define-public python-syllables
+  (package
+    (name "python-syllables")
+    (version "1.0.3")
+    (source (origin
+              (method url-fetch)
+              (uri (pypi-uri "syllables" version))
+              (sha256
+               (base32
+                "0wkl6h0rg6fbsxfp0a8fnibf3l4l6lbh6z12cvcilgb6qhxzpmv3"))))
+    (build-system python-build-system)
+    (home-page "https://github.com/prosegrinder/python-syllables")
+    (synopsis "Package for estimating the number of syllables in a word")
+    (description
+     "This package provides a Python package for estimating the number of
+syllables in a word.")
+    (license license:gpl3)))
+
 (define-public python-sympy
   (package
     (name "python-sympy")
-- 
2.37.2





Information forwarded to guix-patches <at> gnu.org:
bug#57744; Package guix-patches. (Thu, 15 Sep 2022 20:39:02 GMT) Full text and rfc822 format available.

Message #47 received at 57744 <at> debbugs.gnu.org (full text, mbox):

From: M <matf <at> disr.it>
To: 57744 <at> debbugs.gnu.org
Cc: M <matf <at> disr.it>
Subject: [PATCH v3 3/3] gnu: Add shirah.
Date: Thu, 15 Sep 2022 22:38:02 +0200
* gnu/packages/ebook.scm (shirah): New variable.
---
 gnu/packages/ebook.scm | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/gnu/packages/ebook.scm b/gnu/packages/ebook.scm
index 8353012f11..583b3da03b 100644
--- a/gnu/packages/ebook.scm
+++ b/gnu/packages/ebook.scm
@@ -697,3 +697,24 @@ (define-public python-ebooklib
     (description "Ebook library which can handle EPUB2/EPUB3 and Kindle
 format.")
     (license license:agpl3+)))
+
+(define-public shirah
+  (package
+    (name "shirah")
+    (version "1.0.0")
+    (source (origin
+              (method url-fetch)
+              (uri (pypi-uri "shirah_reader" version))
+              (sha256
+               (base32
+                "0j15v435lz68c1mj5clfx5dmfyjc6jvvz2q8hqvv799mb2faj42y"))))
+    (build-system python-build-system)
+    (propagated-inputs (list python-beautifulsoup4 python-ebooklib
+                             python-syllables python-termcolor))
+    (home-page "https://github.com/hallicopter/shirah-reader")
+    (synopsis "Terminal ebook reader with an optional RSVP mode")
+    (description "@command{shirah} is a curses based terminal ebook reader that
+can display ebooks in the usual way or with Rapid Serial Visual Presentation, a
+method to enable speedreading by showing the text word by word at configurable
+speeds.")
+  (license license:gpl2+)))
-- 
2.37.2





Information forwarded to guix-patches <at> gnu.org:
bug#57744; Package guix-patches. (Thu, 15 Sep 2022 20:39:02 GMT) Full text and rfc822 format available.

Message #50 received at 57744 <at> debbugs.gnu.org (full text, mbox):

From: M <matf <at> disr.it>
To: 57744 <at> debbugs.gnu.org
Cc: M <matf <at> disr.it>
Subject: [PATCH v3 1/3] gnu: Add python-ebooklib.
Date: Thu, 15 Sep 2022 22:38:00 +0200
* gnu/packages/ebook.scm (python-ebooklib): New variable.
---
 gnu/packages/ebook.scm | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

diff --git a/gnu/packages/ebook.scm b/gnu/packages/ebook.scm
index c3a27cb113..8353012f11 100644
--- a/gnu/packages/ebook.scm
+++ b/gnu/packages/ebook.scm
@@ -10,6 +10,7 @@
 ;;; Copyright © 2020 Zheng Junjie <873216071 <at> qq.com>
 ;;; Copyright © 2021 la snesne <lasnesne <at> lagunposprasihopre.org>
 ;;; Copyright © 2021 Petr Hodina <phodina <at> protonmail.com>
+;;; Copyright © 2021 Mathieu Laparie <mlaparie <at> disr.it>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -678,3 +679,21 @@ (define-public libmobi
 @item handling encrypted documents
 @end itemize\n")
     (license license:lgpl3+)))
+
+(define-public python-ebooklib
+  (package
+    (name "python-ebooklib")
+    (version "0.17.1")
+    (source (origin
+              (method url-fetch)
+              (uri (pypi-uri "EbookLib" version))
+              (sha256
+               (base32
+                "1w972g0kmh9cdxf3kjr7v4k99wvv4lxv3rxkip39c08550nf48zy"))))
+    (build-system python-build-system)
+    (propagated-inputs (list python-lxml python-six))
+    (home-page "https://github.com/aerkalov/ebooklib")
+    (synopsis "Ebook library which can handle EPUB2/EPUB3 and Kindle format")
+    (description "Ebook library which can handle EPUB2/EPUB3 and Kindle
+format.")
+    (license license:agpl3+)))
-- 
2.37.2





Information forwarded to guix-patches <at> gnu.org:
bug#57744; Package guix-patches. (Thu, 15 Sep 2022 20:42:01 GMT) Full text and rfc822 format available.

Message #53 received at 57744 <at> debbugs.gnu.org (full text, mbox):

From: Mathieu <matf <at> disr.it>
To: M <matf <at> disr.it>
Cc: 57744 <at> debbugs.gnu.org
Subject: Re: [PATCH v3 3/3] gnu: Add shirah.
Date: Thu, 15 Sep 2022 22:41:38 +0200
Thanks for clarifying! Shirah does not include a license file indeed, but the end of the README clarifies the intent of the author: they say we can use gpl2 or any later version at our convenience. "gpl2+" indeed looks more relevant then since it is listed in one of the source files, as you found.

Hopefully the patchset v3 is okay (I had to reset and restart from scratch, I could not figure out how to rebase v3 into three clean commits if starting from v2).

On 2022-09-15 22:38 M <matf <at> disr.it> wrote:

> * gnu/packages/ebook.scm (shirah): New variable.
> ---
> gnu/packages/ebook.scm | 21 +++++++++++++++++++++
> 1 file changed, 21 insertions(+)
>  
> diff --git a/gnu/packages/ebook.scm b/gnu/packages/ebook.scm
> index 8353012f11..583b3da03b 100644
> --- a/gnu/packages/ebook.scm
> +++ b/gnu/packages/ebook.scm
> @@ -697,3 +697,24 @@ (define-public python-ebooklib
> (description "Ebook library which can handle EPUB2/EPUB3 and Kindle
> format.")
> (license license:agpl3+)))
> +
> +(define-public shirah
> +  (package
> +    (name "shirah")
> +    (version "1.0.0")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (pypi-uri "shirah_reader" version))
> +              (sha256
> +               (base32
> +                
> "0j15v435lz68c1mj5clfx5dmfyjc6jvvz2q8hqvv799mb2faj42y"))))
> +    (build-system python-build-system)
> +    (propagated-inputs (list python-beautifulsoup4 python-ebooklib
> +                             python-syllables python-termcolor))
> +    (home-page "https://github.com/hallicopter/shirah-reader")
> +    (synopsis "Terminal ebook reader with an optional RSVP mode")
> +    (description "@command{shirah} is a curses based terminal ebook
> reader that
> +can display ebooks in the usual way or with Rapid Serial Visual
> Presentation, a
> +method to enable speedreading by showing the text word by word at
> configurable
> +speeds.")
> +  (license license:gpl2+)))
> --  
> 2.37.2
>




Reply sent to Christopher Baines <mail <at> cbaines.net>:
You have taken responsibility. (Fri, 16 Sep 2022 10:45:02 GMT) Full text and rfc822 format available.

Notification sent to M <matf <at> disr.it>:
bug acknowledged by developer. (Fri, 16 Sep 2022 10:45:02 GMT) Full text and rfc822 format available.

Message #58 received at 57744-done <at> debbugs.gnu.org (full text, mbox):

From: Christopher Baines <mail <at> cbaines.net>
To: Mathieu <matf <at> disr.it>
Cc: 57744-done <at> debbugs.gnu.org
Subject: Re: [bug#57744] [PATCH v3 3/3] gnu: Add shirah.
Date: Fri, 16 Sep 2022 12:43:27 +0200
[Message part 1 (text/plain, inline)]
Mathieu <matf <at> disr.it> writes:

> Thanks for clarifying! Shirah does not include a license file indeed,
> but the end of the README clarifies the intent of the author: they say
> we can use gpl2 or any later version at our convenience. "gpl2+"
> indeed looks more relevant then since it is listed in one of the
> source files, as you found.
>
> Hopefully the patchset v3 is okay (I had to reset and restart from
> scratch, I could not figure out how to rebase v3 into three clean
> commits if starting from v2).

Thanks, I've pushed these three patches to master now as
eb9a39c1b75a60fe3946496bb2eee8f32dbf09cd.

Thanks again,

Chris
[signature.asc (application/pgp-signature, inline)]

bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Fri, 14 Oct 2022 11:24:07 GMT) Full text and rfc822 format available.

This bug report was last modified 1 year and 166 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.