Re: [PATCH 1/4] emacs: Factorize searching for Emacs packages.

[Ludovic Courtès]

Alex Kost <address@hidden> skribis:

> * emacs/guix-emacs.el: (guix-emacs-directories): New procedure.
> (guix-emacs-find-autoloads-in-directory): Rename to ...
> (guix-emacs-find-autoloads): ... this.
> (guix-emacs-load-autoloads): Replace with ...
> (guix-emacs-autoload-packages): this.  New procedure.  At first, find a
> list of directories with Emacs packages, then add them to 'load-path'
> and finally, load autoloads.
> (guix-emacs-load-autoloads-maybe): Rename to ...
> (guix-emacs-autoload-packages-maybe): ... this.
> * emacs/guix-backend.el (guix-after-repl-operation-hook): Adjust for the
> renaming.
> * doc/emacs.texi (Emacs Initial Setup): Likewise.

[...]

> +(defun guix-emacs-directories (&optional profile)
> +  "Return a list of directories with Emacs packages installed in PROFILE.

Maybe: “Return the list of directories under PROFILE that contain Emacs
packages.  This includes both `share/emacs/site-lisp/guix.d/PACKAGE'
sub-directories and `share/emacs/site-lisp' itself.”

Otherwise LGTM.

Ludo’.