unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: Mathieu <matf@disr.it>
To: Christopher Baines <mail@cbaines.net>
Cc: 57744@debbugs.gnu.org
Subject: [bug#57744] [PATCH 0/3] Add shirah and dependencies
Date: Thu, 15 Sep 2022 21:57:42 +0200	[thread overview]
Message-ID: <ab4d17da-afb6-423e-9ccb-4f08f731b18f@disroot.org> (raw)
In-Reply-To: <87mtb28jfe.fsf@cbaines.net>

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@cbaines.net> wrote:

>  
> M <matf@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




  reply	other threads:[~2022-09-15 19:58 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-09-12  0:09 [bug#57744] [PATCH 0/3] Add shirah and dependencies M
2022-09-12  0:10 ` [bug#57744] [PATCH 1/3] gnu: Add python-ebooklib M
2022-09-12  0:10   ` [bug#57744] [PATCH 2/3] gnu: Add python-syllables M
2022-09-12  0:10   ` [bug#57744] [PATCH 3/3] gnu: Add shirah M
2022-09-14  8:09 ` [bug#57744] [PATCH 0/3] Add shirah and dependencies Christopher Baines
2022-09-15 19:57   ` Mathieu [this message]
2022-09-15 19:59     ` Maxime Devos
2022-09-15 19:58 ` [bug#57744] [PATCH v2 1/3] gnu: Add shirah M
2022-09-15 19:58   ` [bug#57744] [PATCH v2 2/3] " M
2022-09-15 20:01     ` Maxime Devos
2022-09-15 20:03       ` Mathieu
2022-09-15 20:17         ` Maxime Devos
2022-09-15 19:58   ` [bug#57744] [PATCH v2 3/3] gnu: Add python-syllables M
2022-09-15 20:38 ` [bug#57744] [PATCH v3 1/3] gnu: Add python-ebooklib M
2022-09-15 20:38   ` [bug#57744] [PATCH v3 2/3] gnu: Add python-syllables M
2022-09-15 20:38   ` [bug#57744] [PATCH v3 3/3] gnu: Add shirah M
2022-09-15 20:41     ` Mathieu
2022-09-16 10:43       ` bug#57744: " Christopher Baines

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ab4d17da-afb6-423e-9ccb-4f08f731b18f@disroot.org \
    --to=matf@disr.it \
    --cc=57744@debbugs.gnu.org \
    --cc=mail@cbaines.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).