Re: [PATCH 7/7] emacs: Add 'guix-devel-build-current-package-definition'.

[Ludovic Courtès]

Alex Kost <address@hidden> skribis:

> Ludovic Courtès (2015-08-18 17:17 +0300) wrote:
>
>> Alex Kost <address@hidden> skribis:
>>
>>> Suggested by Ludovic Courtès <address@hidden>.
>>>
>>> * emacs/guix-guile.el (guix-guile-definition-regexp): New variable.
>>>   (guix-guile-current-definition): New function.
>>> * emacs/guix-devel.el (guix-devel-build-current-package-definition): New
>>>   command.
>>> * doc/emacs.texi (Emacs Development): Document it.
>>
>> The subject line is incorrect.
>
> Ah, indeed!  It should be:
>
>   emacs: Add 'guix-devel-build-current-package-definition'.
>
> Is that OK?

Yes.

>> Also, what about suggesting or even defining a key binding for this one?
>> It sounds very useful.
>
> There is an example of defining a key binding for a command in this
> section.  As for the real defining, not sure if it is worth, but yeah,
> why not.  We may define 'guix-minor-mode', for example, with
> guix-specific key bindings.  But I don't know what bindings to use:
> 'geiser-mode' already binds many commands to [C-c …].  Perhaps to put
> the additional keys in [C-c g …]?

Yes, ‘guix-minor-mode’ and C-c g both sound like good ideas to me.

Thank you,
Ludo’.