Re: [PATCH 2/2] builtin/symbolic-ref.c: add option to output shortened ref
- From
Jan Krüger <jk@jk.gs>
- Date
- Feb 27, 2012, 22:46 UTC
- Message-ID
- <4F4C07AD.3050404@jk.gs>
- In-Reply-To
- <7vwr7752s8.fsf@alter.siamese.dyndns.org>
On 02/27/2012 11:28 PM, Junio C Hamano wrote:
Show 7 quoted lines
> I think this one (unlike 1/2) makes sense, but a single letter -s feels a > bit too vague. Always spelling in long option "--short" so that it > matches "%(refname:short)" in for-each-ref might be better, I would think. > > Especially given that the expected use case is primarily in scripts not > from the command line, being more explicit and easier to read has value > over being short and easier to (mis)type.
Good point. If we can agree on what to do with the first patch, I'll change that in the next iteration.
-Jan