Re: [PATCH 6/6] misc: update --help content again

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 06/26/2017 02:29 PM, Ruediger Meier wrote:
> From: Ruediger Meier <ruediger.meier@xxxxxxxxxxx>
> 
> We change
> 
>  -h, --help     display this help and exit
>  -V, --version  output version information and exit
> 
> to
> 
>  -h, --help     display this help
>  -V, --version  print version
> 
> Some thoughts about this:
> 
>   * use "display" for --help because it matches better if we
>     would add pager support (like git --help)
>   * "print" for --version to be different

To be different why? It makes it sound like help and version will not
have the same action. Laymen think 'print' means send to the printer.
Translators will have to figure out which different words to use.

What is broken by:

 -h, --help     display this help
 -V, --version  display version

I really think it is a  good idea to use consistent language for
comparable actions. It is easier to comprehend, especially for
non-native speakers. Once they understand the meaning of a term,
possibly from a context that they the understand well, then that
knowledge is transferable to other contexts. Why add confusion by
constantly interchanging term synonyms?


>   * "this" for --help is important to make clear that running
>     --help would not give you any better information than the
>     one you see already
>   * remove "information and exit" because it's bloat for the
>     short-help, everybody knows what it does if it exists
> 
> In the manpages we should use the old, longer but more correct
> descriptions, inclusive a reminder if --help/--version are only
> working when used as the only option. Note the term "version
> information" indicates that we don't only print a single version
> number.
> 
> CC: J William Piggott <elseifthen@xxxxxxx>
> Signed-off-by: Ruediger Meier <ruediger.meier@xxxxxxxxxxx>
> ---
>  include/c.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/include/c.h b/include/c.h
> index ce5551535..be7e6fd53 100644
> --- a/include/c.h
> +++ b/include/c.h
> @@ -316,8 +316,8 @@ static inline int xusleep(useconds_t usec)
>  #define USAGE_COLUMNS    _("\nAvailable output columns:\n")
>  #define USAGE_SEPARATOR    "\n"
>  
> -#define USAGE_HELP_TXT    _("display this help and exit")
> -#define USAGE_VERSION_TXT _("output version information and exit")
> +#define USAGE_HELP_TXT    _("display this help")
> +#define USAGE_VERSION_TXT _("print version")
>  
>  #define PRINT_USAGE_HELP(marg_dsc) \
>  	printf( \
> 
--
To unsubscribe from this list: send the line "unsubscribe util-linux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux