[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 4/7] emacs: Add 'guix-devel-use-current-module'.
From: |
Ludovic Courtès |
Subject: |
Re: [PATCH 4/7] emacs: Add 'guix-devel-use-current-module'. |
Date: |
Tue, 18 Aug 2015 16:05:40 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Alex Kost <address@hidden> skribis:
> * emacs/guix-guile.el (guix-guile-current-module): New function.
> * emacs/guix-devel.el: New file.
> (guix-devel-use-modules, guix-devel-use-current-module): New functions.
> * doc/emacs.texi (Emacs Development): New node. Document
> 'guix-devel-use-current-module'.
[...]
> address@hidden Emacs Development
> address@hidden Development
> +
> +This section describes some additional commands for Guix developers who
> +use Geiser to work with guile code.
s/guile/Guile/
It’s fine to leave this section here, but there should be a sentence
under “Contributing” with a cross-reference to here.
> address@hidden @kbd
> +
> address@hidden M-x guix-devel-use-current-module
> +Use the current guile module. Often after opening a scheme file, you
Capitalize “Guile” and “Scheme” here.
> +want to use a module it defines, so you switch to the Geiser REPL and
> +write @code{,use (some module)} there. You may just use this command
> +instead.
Could you mention a use case here? Like:
For instance, you may call this command while visiting a file that
defines packages; you can then switch to the associated REPL with
@kbd{C-c C-z}, which will be in the current module, which is handy.
The “-devel” part might make it sound like it has an effect similar to
‘pre-inst-env’ though. Perhaps we should explicitly remind developers
that they must set ‘guix-load-path’ appropriately?
Otherwise OK, thanks!
Ludo’.
- Re: [PATCH 4/7] emacs: Add 'guix-devel-use-current-module'.,
Ludovic Courtès <=