[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/4] emacs: Move guix-guile-program to 'guix-config'.
From: |
Alex Kost |
Subject: |
Re: [PATCH 3/4] emacs: Move guix-guile-program to 'guix-config'. |
Date: |
Fri, 07 Aug 2015 10:21:39 +0300 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Thank you, all 3 patches look good to me. I think you can commit them.
Mathieu Lirzin (2015-08-06 20:55 +0300) wrote:
> diff --git a/emacs/guix-config.el.in b/emacs/guix-config.el.in
> index 8804f72..0ab97e2 100644
> --- a/emacs/guix-config.el.in
> +++ b/emacs/guix-config.el.in
> @@ -22,6 +22,15 @@
> (defconst guix-emacs-interface-directory
> (replace-regexp-in-string "${prefix}" "@prefix@" "@emacsuidir@"))
>
> +(defvar guix-guile-program "@GUILE@"
> + "Name of the guile executable used for Guix REPL.
> +May be either a string (the name of the executable) or a list of
> +strings of the form:
> +
> + (NAME . ARGS)
> +
> +Where ARGS is a list of arguments to the guile program.")
> +
> (defconst guix-state-directory
> ;; This must match `NIX_STATE_DIR' as defined in `daemon.am'.
> (or (getenv "NIX_STATE_DIR") "@guix_localstatedir@/guix"))
I would rather add 'guix-guile-program' after both 'guix-…-directory'
constants, but it doesn't really matter.
--
Alex