From: Oleg Pykhalov <go.wigust@gmail.com>
To: Jack Hill <jackhill@jackhill.us>
Cc: 35919-done@debbugs.gnu.org
Subject: bug#35919: recutils package does not install emacs rec-mode
Date: Mon, 10 Jun 2019 07:09:54 +0300 [thread overview]
Message-ID: <87zhmqaxnx.fsf@gmail.com> (raw)
In-Reply-To: <alpine.DEB.2.20.1906092149020.5164@marsh.hcoop.net> (Jack Hill's message of "Sun, 9 Jun 2019 22:01:12 -0400 (EDT)")
[-- Attachment #1: Type: text/plain, Size: 1456 bytes --]
Jack Hill <jackhill@jackhill.us> writes:
> On Sun, 9 Jun 2019, Oleg Pykhalov wrote:
>
>> WDYT about creating a new package recipe in
>> ‘gnu/packages/emacs-xyz.scm’, which inherits ‘recutils’ and sets build
>> system to ‘emacs-build-system’? We will probably need to change
>> directory after unpack phase in this case, but we will not need to
>> maintain Emacs lisp files compilation manually.
>
> I have prepared the following patch implementing this suggestion
> (thanks again, I'm much happier with this). I ended up putting it in
> databases.scm rather than emacs-xyz.scm so that it can live with the
> recutls definition, and not run afoul of the advice by Ludo’ in [0],
> "The lesson is: when inheriting from a package, do it in the module
> where it’s defined."
> [0] https://issues.guix.gnu.org/issue/35154#5
Ah, sorry about that - I forgot this.
> I have also set the inputs and native inputs to the empty lists,
> because as far as I can tell, the emacs mode does not depend on
> anything outside of the emacs-build-system.
OK, I fixed a typo in emacs-recutils synopsis and formatted Git
messages. You could look for messages examples with a handy command
‘vc-print-log’ in Emacs which is binded to ‘C-x v l’ by default. Also I
modified a description a little bit for better searching if you don't
mind. Pushed as 535a69e9e5aae2f59470caf900ece56f83ee6e36 to master.
Thanks,
Oleg.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]
next prev parent reply other threads:[~2019-06-10 4:11 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-05-26 18:55 bug#35919: recutils package does not install emacs rec-mode Jack Hill
2019-05-28 15:16 ` Ludovic Courtès
2019-05-28 21:00 ` Jack Hill
2019-06-08 19:08 ` Jack Hill
2019-06-08 21:07 ` Jack Hill
2019-06-09 18:06 ` Oleg Pykhalov
2019-06-09 18:55 ` Jack Hill
2019-06-10 2:01 ` Jack Hill
2019-06-10 4:09 ` Oleg Pykhalov [this message]
2019-06-10 17:30 ` Jack Hill
2019-06-10 2:02 ` bug#35919: [PATCH] gnu: Add emacs-recutils Jack Hill
2019-06-10 3:24 ` bug#35919: recutils package does not install emacs rec-mode Jack Hill
2019-06-10 3:25 ` bug#35919: [PATCH] gnu: recutils: remove emacs-minimal native input Jack Hill
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=87zhmqaxnx.fsf@gmail.com \
--to=go.wigust@gmail.com \
--cc=35919-done@debbugs.gnu.org \
--cc=jackhill@jackhill.us \
/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).