[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#10253: mention +FORMAT in ls time style reminder help blurb
From: |
Paul Eggert |
Subject: |
bug#10253: mention +FORMAT in ls time style reminder help blurb |
Date: |
Sun, 11 Dec 2011 14:48:20 -0800 |
User-agent: |
Mozilla/5.0 (X11; Linux i686; rv:8.0) Gecko/20111124 Thunderbird/8.0 |
I like the change, thanks. A couple of nits:
On 12/11/11 03:07, Jim Meyering wrote:
> + fprintf (stderr,
> + _(" - `+' followed by a date format string\n"));
I suggest supplying an example and quoting "date" so that it's clearer
that it's talking about the `date' command. Something like this, perhaps?
_(" - +FORMAT (e.g., +%H:%M) for a `date'-style format\n")
> + fprintf (stderr, " - `[posix-]%s'\n", *p++);
I suggest removing the ` and ' since they are locale-dependent
and aren't needed here (plus, that works better with the above
suggestion....).
> + fprintf (stderr, _("Valid arguments are:"));
Isn't the usual style to use fputs when there's no directive
in the format? There's one other example of this.
bug#10253: mention +FORMAT in ls time style reminder help blurb, jidanni, 2011/12/10
bug#10253: mention +FORMAT in ls time style reminder help blurb, jidanni, 2011/12/11
bug#10253: mention +FORMAT in ls time style reminder help blurb, jidanni, 2011/12/12
bug#10253: mention +FORMAT in ls time style reminder help blurb, jidanni, 2011/12/12