Ludovic Courtès (2016-06-21 23:41 +0300) wrote: > Ricardo Wurmus <[email protected]> skribis: > >> Alex Kost <[email protected]> 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
