From: Alex Kost <alezost@gmail.com>
To: guix-devel@gnu.org
Subject: Re: [PATCH] guix: Clean up --help messages.
Date: Sun, 07 Jun 2015 22:26:13 +0300 [thread overview]
Message-ID: <87ioazb9re.fsf@gmail.com> (raw)
In-Reply-To: <87r3pnub6p.fsf@gmail.com> (Alex Kost's message of "Sun, 07 Jun 2015 12:20:30 +0300")
[-- Attachment #1: Type: text/plain, Size: 1069 bytes --]
Sorry, there is another thing I forgot. Currently ‘guix package --help’
displays the following:
--8<---------------cut here---------------start------------->8---
Usage: guix package [OPTION]... PACKAGES...
Install, remove, or upgrade PACKAGES in a single transaction.
-i, --install=PACKAGE install PACKAGE
-e, --install-from-expression=EXP
install the package EXP evaluates to
-r, --remove=PACKAGE remove PACKAGE
...
--8<---------------cut here---------------end--------------->8---
I think it is not clear that multiple packages may be installed/removed
by a single command. If it would look like this:
-i, --install=PACKAGE... install PACKAGEs
it may not be clear how the PACKAGEs are separated, since it is a common
practice to use "," to separate things after an "=" sign. So I suggest
to remove "=" from ‘--install’ and ‘--remove’ options:
-i, --install PACKAGE ... install PACKAGEs
In my opinion it is the most clear way for that. WDYT?
The modified patch is attached.
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-guix-Clean-up-help-messages.patch --]
[-- Type: text/x-diff, Size: 4169 bytes --]
From 58cf074945b1c20e3badb86bf2ed222f89b2c879 Mon Sep 17 00:00:00 2001
From: Alex Kost <alezost@gmail.com>
Date: Sun, 7 Jun 2015 11:46:06 +0300
Subject: [PATCH] guix: Clean up --help messages.
* guix/scripts/import.scm (show-help): Add newline before a list of
importers.
* guix/scripts/lint.scm (show-help): Split a long description line.
* guix/scripts/system.scm (show-help): Format actions the same way as
guix commands and importers are formatted.
---
guix/scripts/import.scm | 1 +
guix/scripts/lint.scm | 3 ++-
guix/scripts/package.scm | 10 ++++++----
guix/scripts/system.scm | 13 +++++++------
4 files changed, 16 insertions(+), 11 deletions(-)
diff --git a/guix/scripts/import.scm b/guix/scripts/import.scm
index 06b4c17..45ce092 100644
--- a/guix/scripts/import.scm
+++ b/guix/scripts/import.scm
@@ -86,6 +86,7 @@ rather than \\n."
Run IMPORTER with ARGS.\n"))
(newline)
(display (_ "IMPORTER must be one of the importers listed below:\n"))
+ (newline)
(format #t "~{ ~a~%~}" importers)
(display (_ "
-h, --help display this help and exit"))
diff --git a/guix/scripts/lint.scm b/guix/scripts/lint.scm
index 3b139ce..3740b71 100644
--- a/guix/scripts/lint.scm
+++ b/guix/scripts/lint.scm
@@ -575,7 +575,8 @@ descriptions maintained upstream."
(define (show-help)
(display (_ "Usage: guix lint [OPTION]... [PACKAGE]...
-Run a set of checkers on the specified package; if none is specified, run the checkers on all packages.\n"))
+Run a set of checkers on the specified package; if none is specified,
+run the checkers on all packages.\n"))
(display (_ "
-c, --checkers=CHECKER1,CHECKER2...
only run the specificed checkers"))
diff --git a/guix/scripts/package.scm b/guix/scripts/package.scm
index 4ae5114..d9f38fb 100644
--- a/guix/scripts/package.scm
+++ b/guix/scripts/package.scm
@@ -422,15 +422,17 @@ ENTRIES, a list of manifest entries, in the context of PROFILE."
(substitutes? . #t)))
(define (show-help)
- (display (_ "Usage: guix package [OPTION]... PACKAGES...
-Install, remove, or upgrade PACKAGES in a single transaction.\n"))
+ (display (_ "Usage: guix package [OPTION]...
+Install, remove, or upgrade packages in a single transaction.\n"))
(display (_ "
- -i, --install=PACKAGE install PACKAGE"))
+ -i, --install PACKAGE ...
+ install PACKAGEs"))
(display (_ "
-e, --install-from-expression=EXP
install the package EXP evaluates to"))
(display (_ "
- -r, --remove=PACKAGE remove PACKAGE"))
+ -r, --remove PACKAGE ...
+ remove PACKAGEs"))
(display (_ "
-u, --upgrade[=REGEXP] upgrade all the installed packages matching REGEXP"))
(display (_ "
diff --git a/guix/scripts/system.scm b/guix/scripts/system.scm
index b6d7d0d..f7c9d83 100644
--- a/guix/scripts/system.scm
+++ b/guix/scripts/system.scm
@@ -368,18 +368,19 @@ boot directly to the kernel or to the bootloader."
Build the operating system declared in FILE according to ACTION.\n"))
(newline)
(display (_ "The valid values for ACTION are:\n"))
+ (newline)
(display (_ "\
- - 'reconfigure', switch to a new operating system configuration\n"))
+ reconfigure switch to a new operating system configuration\n"))
(display (_ "\
- - 'build', build the operating system without installing anything\n"))
+ build build the operating system without installing anything\n"))
(display (_ "\
- - 'vm', build a virtual machine image that shares the host's store\n"))
+ vm build a virtual machine image that shares the host's store\n"))
(display (_ "\
- - 'vm-image', build a freestanding virtual machine image\n"))
+ vm-image build a freestanding virtual machine image\n"))
(display (_ "\
- - 'disk-image', build a disk image, suitable for a USB stick\n"))
+ disk-image build a disk image, suitable for a USB stick\n"))
(display (_ "\
- - 'init', initialize a root file system to run GNU.\n"))
+ init initialize a root file system to run GNU.\n"))
(show-build-options-help)
(display (_ "
--
2.4.2
next prev parent reply other threads:[~2015-06-07 19:26 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-07 9:20 [PATCH] guix: Clean up --help messages Alex Kost
2015-06-07 15:37 ` Ludovic Courtès
2015-06-07 19:26 ` Alex Kost [this message]
2015-06-07 19:37 ` Ludovic Courtès
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87ioazb9re.fsf@gmail.com \
--to=alezost@gmail.com \
--cc=guix-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.