[debbugs-tracker] bug#28365: closed ([PATCH] gnu: emacs-ess: Include the etc directory in the build)

[GNU bug Tracking System]

Your message dated Thu, 07 Sep 2017 10:05:31 +0200
with message-id <address@hidden>
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
address@hidden)


-- 
28365: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=28365
GNU Bug Tracking System
Contact address@hidden with problems

--- Begin Message --- Subject: [PATCH] gnu: emacs-ess: Include the etc directory in the build Date: Wed, 06 Sep 2017 12:20:41 +0200 User-agent: mu4e 0.9.18; emacs 25.2.1

0001-gnu-emacs-ess-Include-the-etc-directory-in-the-build.patch
Description: Text Data

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

--- End Message ---

> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: [bug#28365] [PATCH] gnu: emacs-ess: Include the etc directory in the build
>
>
>
>
> Date: 
>
> Thu, 07 Sep 2017 10:05:31 +0200
>
>
>
>
> User-agent: 
>
> mu4e 0.9.18; emacs 25.1.1
>
>
>
>
> Kyle Meyer writes:
>
> > Roel Janssen <address@hidden> 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
>
>
> --- End Message ---