From: Jan Krüger <jk@jk.gs>
Date: Mon, 27 Feb 2012 22:46:05 GMT
Subject: Re: [PATCH 2/2] builtin/symbolic-ref.c: add option to output shortened ref
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:
> 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