Re: [PATCH 5/6] emacs: Add interface for package locations.

[Ludovic Courtès]

Alex Kost <address@hidden> skribis:

> * emacs/guix-main.scm (%package-location-param-alist): New variable.
> (package-location->sexp, package-location-entries): New procedures.
> * emacs/guix-ui-location.el: New file.
> * emacs.am (ELFILES): Add it.
> * doc/emacs.texi (Emacs Locations): Document 'guix-locations'.
> * NEWS: Mention it.
> ---
>  NEWS                      |  6 ++++
>  doc/emacs.texi            |  7 ++++
>  emacs.am                  |  1 +
>  emacs/guix-main.scm       | 12 +++++++
>  emacs/guix-ui-location.el | 83 
> +++++++++++++++++++++++++++++++++++++++++++++++
>  5 files changed, 109 insertions(+)
>  create mode 100644 emacs/guix-ui-location.el

That it takes so few lines to implement this is a good sign.  :-)

> +*** New Emacs interface for package locations: M-x guix-locations

s/for package locations/to navigate package definitions/ ?

> address@hidden M-x guix-locations
> +Display a list of package locations.  You can press @key{RET} there to

Maybe “Display the list of source files that define packages.”?

Otherwise LGTM, thanks!

Ludo’.