GNU bug report logs - #28365
[PATCH] gnu: emacs-ess: Include the etc directory in the build

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Roel Janssen <roel <at> gnu.org>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#28365: closed ([PATCH] gnu: emacs-ess: Include the etc
 directory in the build)
Date: Thu, 07 Sep 2017 08:06:02 +0000

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

Your message dated Thu, 07 Sep 2017 10:05:31 +0200
with message-id <87pob3q6hf.fsf <at> gnu.org>
and subject line Re: [bug#28365] [PATCH] gnu: emacs-ess: Include the etc directory in the build
has caused the debbugs.gnu.org bug report #28365,
regarding [PATCH] gnu: emacs-ess: Include the etc directory in the build
to be marked as done.

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


-- 
28365: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=28365
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Roel Janssen <roel <at> gnu.org>
To: guix-patches <guix-patches <at> gnu.org>
Subject: [PATCH] gnu: emacs-ess: Include the etc directory in the build
Date: Wed, 06 Sep 2017 12:20:41 +0200

[0001-gnu-emacs-ess-Include-the-etc-directory-in-the-build.patch (text/x-patch, attachment)]

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

Dear Guix,

When loading ESS "the standard way":
  (require 'ess-site)

it fails to load because it cannot find the "etc" directory that should
ship with this package.

This patch puts that directory and its contents (as generated by the
build system) into $PROFILE/share/emacs/site-lisp/guix.d/ess/etc.

After this change, I was able to load ESS "the standard way".

Thanks for your time.

Kind regards,
Roel Janssen

[Message part 5 (message/rfc822, inline)]

From: Roel Janssen <roel <at> gnu.org>
To: Kyle Meyer <kyle <at> kyleam.com>
Cc: 28365-done <at> debbugs.gnu.org
Subject: Re: [bug#28365] [PATCH] gnu: emacs-ess: Include the etc directory in
 the build
Date: Thu, 07 Sep 2017 10:05:31 +0200

Kyle Meyer writes:

> Roel Janssen <roel <at> gnu.org> writes:
>
>> When loading ESS "the standard way":
>>   (require 'ess-site)
>>
>> it fails to load because it cannot find the "etc" directory that should
>> ship with this package.
>
> Thanks, I hit this too.  Here's a fix that I have locally but hadn't
> sent yet.
>
> -- >8 --
> diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
> index 9c0f9bc89..a47e90a40 100644
> --- a/gnu/packages/emacs.scm
> +++ b/gnu/packages/emacs.scm
> @@ -3206,7 +3206,14 @@ E-Prime forbids the use of the \"to be\" form to strengthen your writing.")
>                                    version ".tgz"))
>                (sha256
>                 (base32
> -                "0w7mbbajn377gdmvnd21mpyr368b2ia46gq6cb99y4y5rspf9pcg"))))
> +                "0w7mbbajn377gdmvnd21mpyr368b2ia46gq6cb99y4y5rspf9pcg"))
> +              (modules '((guix build utils)))
> +              (snippet
> +               ;; Add an entry to ess-etc-directory-list that is relative to
> +               ;; guix.d/.
> +               '(substitute* "lisp/ess-site.el"
> +                  (("^  '\\(\"\\.\\./etc/ess/\"" line)
> +                   (string-append line " \"../../../etc/ess/\""))))))
>      (build-system gnu-build-system)
>      (arguments
>       `(#:tests? #f ; There is no test suite.
> -- >8 --
>
> That teaches ess-etc-directory-list about $PROFILE/share/emacs/etc/.  I
> prefer your solution because I think it's less fragile.
>
> I just have a minor comment on the patch description.
>
>> Subject: [PATCH] gnu: emacs-ess: Include the etc directory in the build
>>  output.
>>
>> * gnu/packages/emacs.scm (emacs-ess):  Also copy the etc directory.
>
> That doesn't seem quite right because the etc directory is already
> included in the build; it's under $PROFILE/share/emacs/etc/.  Perhaps
> this should instead say something like "Relocate the etc directory so
> that ESS can find it."

You're right.  I didn't know that the directory was at
$PROFILE/share/emacs/etc/.

So I adjusted the commit message and pushed in
eaff063aa0f478f23b78c5db8ba0d465b3f691cd.

Thanks!

Kind regards,
Roel Janssen

Previous Next

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