Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH 2/2] date: make "local" orthogonal to date format

From: Junio C Hamano <hidden>
Date: 2016-06-15 23:06:25

John Keeping [off-list ref] writes:
On Wed, Sep 02, 2015 at 05:30:14PM -0400, Jeff King wrote:
quoted
I think the error message would be a lot nicer if we indicate that "-"
is syntactically interesting, and say:

  fatal: unknown date-mode modifier: locale
I wonder if we'd be better just saying:

	fatal: unknown date format: short-locale

I'm not sure users will consider "local" to be a modifier, there is
simply a list of formats that happens to include pairs of matching
"-local" and "not -local" variants.
Either is acceptable and better than the 'e' output.  I think yours
is better because it would equally work well for those who think in
terms of "modifier" and those who think in terms of "flat list of
possibilities".

Thanks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help