ng0 (2016-08-08 01:04 +0300) wrote: > Troy Sankey <sankey...@gmail.com> writes: > >> Quoting ng0 (2016-08-07 15:29:34) >>> > (inputs >>> > - `(("emacs" ,emacs) >>> > - ("glib" ,glib) >>> > + `(("glib" ,glib) >>> >>> Why is emacs removed? >> >> Please see the commit message :) >> >> Of course I could be misunderstanding something, but it does build >> successfully. I do not normally use emacs, so I was not able to test >> this particular change.
Commit message: This also removes the emacs input. Quote from the notmuch NEWS file: Drop dependency on "pkg-config emacs". This only means that emacs is not required for notmuch to be built successfully (it is done for those people who don't use emacs). But it doesn't mean this dependency should be dropped. In Guix we usually include all available features. > Ah, sorry. > I don't want to mix up my notmuch and the review, and I'll be occupied > until tuesday night, so someone who does not rely on a functional > notmuch should review if it works correctly. I find it weird that the > input is dropped. You are right, emacs input shouldn't be dropped. This would lead to non-compiled elisp files (I mean /gnu/store/...-notmuch-0.22.1/share/emacs/site-lisp/ would contain only *.el files but not *.elc). I would rather replace 'emacs' with 'emacs-minimal', also I think it should be moved to 'native-inputs'; but this does not relate to this patch (I'll commit this change separately). -- Alex