diff mbox series

[bug#41181] adds emacs-interleave

Message ID 6rxX0vh7GxY4H7d6jXP10G8cqsh8_bs8V4e2JxatbfkiLwsC685g0wInwQSxzRpwsOfhm5aVovpD5lOd55GcNAXVlNGfXu1CYq4ym84-M5c=@protonmail.com
State Accepted
Headers show
Series [bug#41181] adds emacs-interleave | expand

Checks

Context Check Description
cbaines/applying patch fail View Laminar job

Commit Message

guix--- via Guix-patches via May 17, 2020, 1:18 p.m. UTC 
Hello,

Attached is the copyright line that should have been in the previous patch.

Ryan


Sent with ProtonMail Secure Email.

‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Friday, May 15, 2020 4:56 PM, rdes <rdes@protonmail.com> wrote:

> Hello Nicolas,
>
> Thanks for the pointers. Attached is an updated patch.
>
> Ryan
>
> Sent with ProtonMail Secure Email.
>
> ‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
> On Monday, May 11, 2020 3:10 AM, Nicolas Goaziou mail@nicolasgoaziou.fr wrote:
>
> > Hello,
> > rdes via Guix-patches via guix-patches@gnu.org writes:
> >
> > > The attached patch adds the emacs-interleave package.
> >
> > Thank you! Some comments follow.
> >
> > > Subject: [PATCH] gnu: Add emacs-interleave
> >
> > Your commit message should conform to Guix rules. Here, you need to add
> >
> > -   gnu/packages/emacs-xyz.scm (emacs-interleave): New variable.
> >
> > > +(define-public emacs-interleave
> > > +(package
> >
> > Indentation seems wrong.
> >
> > > -   (name "emacs-interleave")
> > >
> > > -   (version "20191129.958")
> > >
> > > -   (source
> > >
> > > -   (origin
> > >
> > > -          (method url-fetch)
> > >
> > >
> > > -          (uri (string-append
> > >
> > >
> > > -                 "https://melpa.org/packages/interleave-"
> > >
> > >
> > > -                 version
> > >
> > >
> > > -                 ".el"))
> > >
> > >
> >
> > We do not use MELPA sources. Instead, you could use `git-fetch' and
> > extract the latest stable release from here:
> > https://github.com/rudolfochrist/interleave/releases
> >
> > > -          (sha256
> > >
> > >
> > > -            (base32
> > >
> > >
> > > -              "0y183ggfmkjjywbcqgyinmr21hsg2vaw4d52bg1gsxbp673za2m8"))))
> > >
> > >
> >
> > Nitpick: could you move the string on the same line as `base32'?
> >
> > > -   (build-system emacs-build-system)
> > > -   (home-page
> > > -   "https://github.com/rudolfochrist/interleave")
> >
> > Ditto. Could you move the string on the line above?
> >
> > > -   (synopsis "Interleaving text books since 2015")
> >
> > I think this synopsis needs to be reworded. As it is, it may not be
> > useful.
> >
> > > -   (description
> > > -   "Interleave presents your PDF side by side to an [[http://orgmode.org][Org Mode]]
> >
> > Description uses Texinfo syntax, not Org syntax. In this case, however,
> > I think "side by side to an Org buffer" is sufficient.
> >
> > > +buffer with your notes, narrowing down to just those passages that are relevant to
> > > +the particular page in the document viewer.
> > > +")
> >
> > You don't need to end the description with a final newline. It would be
> > nice to expound it a bit. For example, it could be interesting to know
> > this is a minor mode.
> > Don't forget to add a copyright line with your name at the top of the file.
> > Could you send an updated patch?
> > Regards,
> >
> > Nicolas Goaziou
diff mbox series

Patch

From 7cc8535fb92e5d39276ec5cdd5bc078d3d8c55b6 Mon Sep 17 00:00:00 2001
From: Ryan Desfosses <ryan@desfo.org>
Date: Sun, 17 May 2020 09:15:46 -0400
Subject: [PATCH] * gnu/packages/emacs-xyz.scm: Add copyright.

---
 gnu/packages/emacs-xyz.scm | 1 +
 1 file changed, 1 insertion(+)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index b70b9fffb8..5d0beeb2d9 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -70,6 +70,7 @@ 
 ;;; Copyright © 2020 Alberto Eleuterio Flores Guerrero <barbanegra+guix@posteo.mx>
 ;;; Copyright © 2020 Marius Bakke <mbakke@fastmail.com>
 ;;; Copyright © 2020 pinoaffe <pinoaffe@airmail.cc>
+;;; Copyright © 2020 Ryan Desfosses <rdes@protonmail.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
-- 
2.26.2