emacs-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#43635: closed ([PATCH] gnu: Add emacs-sqlite.)


From: GNU bug Tracking System
Subject: bug#43635: closed ([PATCH] gnu: Add emacs-sqlite.)
Date: Thu, 01 Oct 2020 13:43:02 +0000

Your message dated Thu, 01 Oct 2020 15:42:17 +0200
with message-id <87d02212rq.fsf@nicolasgoaziou.fr>
and subject line Re: [bug#43635] hope, this is right
has caused the debbugs.gnu.org bug report #43635,
regarding [PATCH] gnu: Add emacs-sqlite.
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
43635: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=43635
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems
--- Begin Message --- Subject: [PATCH] gnu: Add emacs-sqlite. Date: Sat, 26 Sep 2020 22:36:58 +0200 (CEST)

Attachment: 0001-gnu-Add-emacs-sqlite.patch
Description: Text Data


--- End Message ---
--- Begin Message --- Subject: Re: [bug#43635] hope, this is right Date: Thu, 01 Oct 2020 15:42:17 +0200 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux)
Hello,

Adam Kandur via Guix-patches via <guix-patches@gnu.org> writes:

> Subject: [PATCH] gnu: Add emacs-sqlite.el

Thank you. I applied your patch with the changes detailed below.

> +  (let ((commit "dad42b8bbca4994be1871343dd18fd6528ee5797") ;; there is no 
> tagged commit
> +     (revision "0"))

I moved the comment on the line above since comments starting with two
semicolons are expected to stand by their own line.

> +    (package
> +      (name "emacs-sqlite.el")

I changed the name to "emacs-sqlite". Ditto for the name of the variable.

> +      (synopsis "SQLite Interface for Emas Lisp")

I fixed the type and changed case for "Interface".

> +      (description "@code{sqlite.el} is a simple SqliteInterface for 
> connecting and retrieving
> +information using the SQLite program through Elisp programming.  It
> is  not intended as a user interface.")

I fixed spacing and renamed the package.

I also moved the package elsewhere, since adding a package at the end of
the file is more likely to cause merge conflicts.

Regards,
-- 
Nicolas Goaziou


--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]