Re: [PATCH] build: emacs: Search elisp files in "share/emacs/site-lisp".

[Alex Kost]

Ludovic Courtès (2016-06-21 23:41 +0300) wrote:

> Ricardo Wurmus <address@hidden> skribis:
>
>> Alex Kost <address@hidden> writes:
>>
>>> Ricardo found a problem while working on some emacs package that
>>> requires EMMS: the elisp file(s) of this package is(are) not compiled.
>>>
>>> This happens because currently emacs-build-system adds only
>>> "share/emacs/site-lisp/guix.d/<package>" to emacs load-path when it
>>> compiles elisp files, but some emacs packages (for example, emms) put
>>> their files in "share/emacs/site-lisp", so this directory should also be
>>> added.  The attached patch will do it.
>>
>> Thank you for the patch!  This looks like the right thing to do.
>
> +1

Pushed, thanks.

>> Should we also (in a later patch) add a way to override or append to the
>> list of directories, e.g. via the “arguments“ field?
>
> FWIW I’m not convinced this is necessary, but we can always see later.

I agree: we can add this feature later if it will be needed.  But currently
it is not needed, as all our emacs packages put *.el files either in
"share/emacs/site-lisp" or in "share/emacs/site-lisp/guix.d/<package>".

-- 
Alex