GNU bug report logs - #35919
recutils package does not install emacs rec-mode

Previous Next

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

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

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

From: Jack Hill <jackhill <at> jackhill.us>
To: bug-guix <at> gnu.org
Subject: recutils package does not install emacs rec-mode
Date: Sun, 26 May 2019 14:55:53 -0400 (EDT)

Hello Guix,

Emacs is listed as a native imput of the recutils package, but the emacs 
rec-mode is not install (it is found in the etc directory of the source).

Best,
Jack

jackhill <at> alperton ~$ guix describe
Generation 33	May 23 2019 21:23:03	(current)
  guix e26d628
    repository URL: https://git.savannah.gnu.org/git/guix.git
    branch: master
    commit: e26d628b0fabf5a0aa7c4164a9558c66c61e02ab
jackhill <at> alperton ~$ cd $(guix build recutils)
jackhill <at> alperton /gnu/store/163pczynvjfq4l6p7nam5sx9wbz07bgz-recutils-1.8$ find .
.
./lib
./lib/librec.so.1.0.0
./lib/testrec.so.0
./lib/readrec.so.0
./lib/readrec.so.0.0.0
./lib/testrec.la
./lib/readrec.la
./lib/readrec.so
./lib/librec.so.1
./lib/testrec.so
./lib/librec.la
./lib/testrec.a
./lib/readrec.a
./lib/librec.so
./lib/testrec.so.0.0.0
./lib/librec.a
./share
./share/doc
./share/doc/recutils-1.8
./share/doc/recutils-1.8/COPYING
./share/locale
./share/locale/vi
./share/locale/vi/LC_MESSAGES
./share/locale/vi/LC_MESSAGES/recutils.mo
./share/locale/sr
./share/locale/sr/LC_MESSAGES
./share/locale/sr/LC_MESSAGES/recutils.mo
./share/locale/fi
./share/locale/fi/LC_MESSAGES
./share/locale/fi/LC_MESSAGES/recutils.mo
./share/locale/pt_BR
./share/locale/pt_BR/LC_MESSAGES
./share/locale/pt_BR/LC_MESSAGES/recutils.mo
./share/locale/de
./share/locale/de/LC_MESSAGES
./share/locale/de/LC_MESSAGES/recutils.mo
./share/locale/uk
./share/locale/uk/LC_MESSAGES
./share/locale/uk/LC_MESSAGES/recutils.mo
./share/locale/fr
./share/locale/fr/LC_MESSAGES
./share/locale/fr/LC_MESSAGES/recutils.mo
./share/locale/es
./share/locale/es/LC_MESSAGES
./share/locale/es/LC_MESSAGES/recutils.mo
./share/locale/sv
./share/locale/sv/LC_MESSAGES
./share/locale/sv/LC_MESSAGES/recutils.mo
./share/locale/nl
./share/locale/nl/LC_MESSAGES
./share/locale/nl/LC_MESSAGES/recutils.mo
./share/man
./share/man/man1
./share/man/man1/recdel.1.gz
./share/man/man1/rec2csv.1.gz
./share/man/man1/recfmt.1.gz
./share/man/man1/recinf.1.gz
./share/man/man1/recfix.1.gz
./share/man/man1/recins.1.gz
./share/man/man1/recsel.1.gz
./share/man/man1/recset.1.gz
./share/man/man1/csv2rec.1.gz
./share/info
./share/info/recutils.info.gz
./share/info/rec-mode.info.gz
./share/recutils
./share/recutils/etc
./share/recutils/etc/FSD.rec
./include
./include/rec.h
./bin
./bin/recfmt
./bin/recins
./bin/recfix
./bin/recdel
./bin/rec2csv
./bin/recset
./bin/recinf
./bin/recsel
./bin/csv2rec

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

From: Ludovic Courtès <ludo <at> gnu.org>
To: Jack Hill <jackhill <at> jackhill.us>
Cc: 35919 <at> debbugs.gnu.org
Subject: Re: bug#35919: recutils package does not install emacs rec-mode
Date: Tue, 28 May 2019 17:16:20 +0200

Hi Jack,

Jack Hill <jackhill <at> jackhill.us> skribis:

> Emacs is listed as a native imput of the recutils package, but the
> emacs rec-mode is not install (it is found in the etc directory of the
> source).

That must be a regression.  Could you check how this can be fixed?

Thanks,
Ludo’.

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

From: Jack Hill <jackhill <at> jackhill.us>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 35919 <at> debbugs.gnu.org
Subject: Re: bug#35919: recutils package does not install emacs rec-mode
Date: Tue, 28 May 2019 17:00:54 -0400 (EDT)

[Message part 1 (text/plain, inline)]

On Tue, 28 May 2019, Ludovic Courtès wrote:

> That must be a regression.  Could you check how this can be fixed?

Sure, I can take a look and see if I can spot the problem.

Best,
Jack

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

From: Jack Hill <jackhill <at> jackhill.us>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 35919 <at> debbugs.gnu.org
Subject: Re: bug#35919: recutils package does not install emacs rec-mode
Date: Sat, 8 Jun 2019 15:08:58 -0400 (EDT)

I've looked at this some, and it looks to me like it is a change in the 
upstream build system. I can confirm that the problem arrived in Guix when 
the package version was updated from 1.7 to 1.8. In particular, it looks 
like lispdir is no longer being set by configure. The configure from 1.7 
has

```
  if test "x$lispdir" = x; then
    lispdir='${datarootdir}/emacs/site-lisp'

  fi
```

while the configure from 1.8 does not mention lispdir at all.

I'll ask the recutils upstream if this was intentional.

In the meantime, what do we want to do in Guix? I see two options: one is 
patching the build system, and the other is adding a phase to install the 
elisp files. The latter seems to me like the clearest and easiest 
approach here.

Thoughts?

Best,
Jack

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

From: Jack Hill <jackhill <at> jackhill.us>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 35919 <at> debbugs.gnu.org
Subject: Re: bug#35919: recutils package does not install emacs rec-mode
Date: Sat, 8 Jun 2019 17:07:10 -0400 (EDT)

[Message part 1 (text/plain, inline)]

On Sat, 8 Jun 2019, Jack Hill wrote:

> I'll ask the recutils upstream if this was intentional.

I have done so at: 
<https://lists.gnu.org/archive/html/help-recutils/2019-06/msg00000.html>

> In the meantime, what do we want to do in Guix? I see two options: one is 
> patching the build system, and the other is adding a phase to install the 
> elisp files. The latter seems to me like the clearest and easiest approach 
> here.

The attached patch does the latter, copying the two elisp files to the 
site-elisp directory. It does not byte-compile the files or generate 
autoloads (we do we need to generate autoloads?). However, I don't think 
recutils 1.7 did either of those either, so at least we're no worse off 
then we were then.

What do you think?

Best,
Jack

[0001-gnu-recutils-Install-elisp-files.patch (text/x-diff, attachment)]

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

From: Oleg Pykhalov <go.wigust <at> gmail.com>
To: Jack Hill <jackhill <at> jackhill.us>
Cc: 35919 <at> debbugs.gnu.org
Subject: Re: bug#35919: recutils package does not install emacs rec-mode
Date: Sun, 09 Jun 2019 21:06:16 +0300

[Message part 1 (text/plain, inline)]

Hi Jack,

Jack Hill <jackhill <at> jackhill.us> writes:

[…]

> The attached patch does the latter, copying the two elisp files to the
> site-elisp directory. It does not byte-compile the files or generate
> autoloads (we do we need to generate autoloads?). However, I don't
> think recutils 1.7 did either of those either, so at least we're no
> worse off then we were then.

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.

Regards,
Oleg.

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

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

From: Jack Hill <jackhill <at> jackhill.us>
To: Oleg Pykhalov <go.wigust <at> gmail.com>
Cc: 35919 <at> debbugs.gnu.org
Subject: Re: bug#35919: recutils package does not install emacs rec-mode
Date: Sun, 9 Jun 2019 14:55:37 -0400 (EDT)

[Message part 1 (text/plain, inline)]

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.

Oleg,

I like that idea. I'll give it a go, thanks for the suggestion.

Best,
Jack

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

From: Jack Hill <jackhill <at> jackhill.us>
To: Oleg Pykhalov <go.wigust <at> gmail.com>
Cc: 35919 <at> debbugs.gnu.org
Subject: Re: bug#35919: recutils package does not install emacs rec-mode
Date: Sun, 9 Jun 2019 22:01:12 -0400 (EDT)

[Message part 1 (text/plain, inline)]

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

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.

Best,
Jack

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

From: Jack Hill <jackhill <at> jackhill.us>
To: Oleg Pykhalov <go.wigust <at> gmail.com>
Cc: 35919 <at> debbugs.gnu.org
Subject: [PATCH] gnu: Add emacs-recutils
Date: Sun, 9 Jun 2019 22:02:15 -0400 (EDT)

[Message part 1 (text/plain, inline)]

From 9ef1261d8223eeb4b88da4b9ac8659ca60331ffd Mon Sep 17 00:00:00 2001

* gnu/packages/databases.scm (emacs-recutils) new variable.
---
 gnu/packages/databases.scm | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/gnu/packages/databases.scm b/gnu/packages/databases.scm
index ad4698f24c..30821e330b 100644
--- a/gnu/packages/databases.scm
+++ b/gnu/packages/databases.scm
@@ -33,6 +33,7 @@
 ;;; Copyright © 2018 Amirouche Boubekki <amirouche <at> hypermove.net>
 ;;; Copyright © 2018 Joshua Sierles, Nextjournal <joshua <at> nextjournal.com>
 ;;; Copyright © 2018 Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
+;;; Copyright © 2019 Jack Hill <jackhill <at> jackhill.us>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -108,6 +109,7 @@
   #:use-module (guix download)
   #:use-module (guix bzr-download)
   #:use-module (guix git-download)
+  #:use-module (guix build-system emacs)
   #:use-module (guix build-system gnu)
   #:use-module (guix build-system go)
   #:use-module (guix build-system perl)
@@ -965,6 +967,26 @@ types are supported, as is encryption.")
     (license license:gpl3+)
     (home-page "https://www.gnu.org/software/recutils/")))

+(define-public emacs-recutils
+  (package
+    (inherit recutils)
+    (name "emacs-recutils")
+    (build-system emacs-build-system)
+    (arguments
+     '(#:phases
+       (modify-phases %standard-phases
+         (add-after 'unpack 'change-directory
+           (lambda _
+             (chdir "etc")
+             #t)))))
+    (native-inputs '())
+    (inputs '())
+    (synopsis "Emacs more for working with recutils database files")
+    (description "This package provides rec-mode for Emacs for working with
+GNU Recutils text-based, human-editable databases.  It supports editing,
+navigation, and querying of recutils database files including field and record
+folding.")))
+
 (define-public rocksdb
   (package
     (name "rocksdb")
-- 
2.21.0

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

From: Jack Hill <jackhill <at> jackhill.us>
To: 35919 <at> debbugs.gnu.org
Subject: recutils package does not install emacs rec-mode
Date: Sun, 9 Jun 2019 23:24:43 -0400 (EDT)

I realized that emacs-minimal should also be removed from the 
native-inputs of recutils since it is not being using during the build.

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

From: Jack Hill <jackhill <at> jackhill.us>
To: 35919 <at> debbugs.gnu.org
Subject: [PATCH] gnu: recutils: remove emacs-minimal native input
Date: Sun, 9 Jun 2019 23:25:20 -0400 (EDT)

From 6cb65ab8ae2a0cb239cc476a6859ab407cec30e2 Mon Sep 17 00:00:00 2001

* gnu/packages/databases.scm (recutils)
[native-inputs]: Remove emacs-minimal
---
 gnu/packages/databases.scm | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/gnu/packages/databases.scm b/gnu/packages/databases.scm
index ad4698f24c..0edb3d445c 100644
--- a/gnu/packages/databases.scm
+++ b/gnu/packages/databases.scm
@@ -945,8 +945,7 @@ organized in a hash table or B+ tree.")
                                       (assoc-ref %build-inputs "bash:include")
                                       "/include/bash"))))

-    (native-inputs `(("emacs" ,emacs-minimal)
-                     ("bc" ,bc)
+    (native-inputs `(("bc" ,bc)
                      ("bash:include" ,bash "include")
                      ("check" ,check)
                      ("libuuid" ,util-linux)
-- 
2.21.0

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

From: Oleg Pykhalov <go.wigust <at> gmail.com>
To: Jack Hill <jackhill <at> jackhill.us>
Cc: 35919-done <at> debbugs.gnu.org
Subject: Re: bug#35919: recutils package does not install emacs rec-mode
Date: Mon, 10 Jun 2019 07:09:54 +0300

[Message part 1 (text/plain, inline)]

Jack Hill <jackhill <at> 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.

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

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

From: Jack Hill <jackhill <at> jackhill.us>
To: Oleg Pykhalov <go.wigust <at> gmail.com>
Cc: 35919-done <at> debbugs.gnu.org
Subject: Re: bug#35919: recutils package does not install emacs rec-mode
Date: Mon, 10 Jun 2019 13:30:36 -0400 (EDT)

[Message part 1 (text/plain, inline)]

On Mon, 10 Jun 2019, Oleg Pykhalov wrote:

>> 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.

No worries, I probably remember it more strongly since I was affected by 
the other bug.

> 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 for the tip and for improving the description. I'm still learning 
the art of writing a good description.

Best,
Jack

Previous Next

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