Re: [PATCH] : ls: clarify documentation of exit statuses

[Jim Meyering]

Paul Eggert <address@hidden> wrote:
> Jim Meyering <address@hidden> writes:
>> Good thing I tried to compile before pushing!
>> The above lacks \n\ on the "0 if OK" line.
>
> It's also lacking a close-parenthesis in the text.  Also, that wording
> is a bit verbose for my taste; the verbosity makes it a bit hard to see
> the exit-status pattern.  How about this more-concise version?
>
> 2008-10-16  Paul Eggert  <address@hidden>
>
>       * src/ls.c (usage): Describe exit statuses more concisely.
>
> diff --git a/src/ls.c b/src/ls.c
> index ea35b17..34e43ed 100644
> --- a/src/ls.c
> +++ b/src/ls.c
> @@ -4532,10 +4532,8 @@ colors, and can be set easily by the dircolors 
> command.\n\
>  \n\
>  Exit status:\n\
>   0  if OK,\n\
> - 1  if minor problems (e.g., failure to access a file or directory not\n\
> -    specified as a command line argument,\n\
> - 2  if serious trouble (e.g., memory exhausted, invalid option or failure\n\
> -    to access a file or directory specified as a command line argument).\n\
> + 1  if minor problems (e.g., cannot access subdirectory),\n\
> + 2  if serious trouble (e.g., cannot access command-line argument).\n\
>  "), stdout);
>        emit_bug_reporting_address ();
>      }

I've just pushed that.