[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#22587: ‘guix edit’ & ‘M-x guix-edit' typo, rename, & mode change
From: |
Alex Kost |
Subject: |
bug#22587: ‘guix edit’ & ‘M-x guix-edit' typo, rename, & mode change |
Date: |
Mon, 08 Feb 2016 11:34:47 +0300 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
myglc2 (2016-02-07 21:04 +0300) wrote:
> From guix INFO:
>
> 6.2 Invoking ‘guix edit’
> [...]
> launches the program specified in the ‘VISUAL’ or in the ‘EDITOR’
> environment variable to edit the recipe of GCC 4.8.4 and that of Vim."
>
> TYPO:
>
> "edit" (last line above) should be replaced with "view", "inspect", or
> "examine".
Just to mention - I like "edit" name :-)
> RENAME:
>
> Calling these functions 'guix edit' and 'M-x guix-edit' implies that the
> user will be able to modify the recipe, but this is not actually the
> case. The functions should be given a more informative and accurate
> name, such as: 'guix view', 'guix inspect', or 'guix examine'.
Along with the package recipes that come with Guix, a user can also have
his/her own packages (specified using GUIX_PACKAGE_PATH env var), and
"guix edit my-super-package" opens a user's file with this package. It
is highly likely that this file is editable, so "guix edit" is a perfect
name in this case I think. IMO it's a user responsibility to understand
what files can be edited and what cannot.
But I agree that this may be confusing, so maybe we should clarify the
manual to explain in (info "(guix) Invoking guix edit") that store files
must not be edited.
> MODE CHANGE:
>
> In emacs, read-only .scm files should be opened in view-mode.
I don't agree. For example, when you open "/etc/hosts" (or whatever)
file in Emacs, you get a message:
Note: file is write protected
which indicates that it is opened read-only, but view-mode is not used
by default. You can enable it manually if you wish. The same thing
happens with package files.
--
Alex