Re: [PATCH 4/4] server info: Add HEAD to info/refs

From: Junio C Hamano <junkio@cox.net>
Date: Apr 27, 2007, 04:36 UTC
Message-ID: <7vps5qph73.fsf@assigned-by-dhcp.cox.net>
In-Reply-To: <20070427042802.GW4489@pasky.or.cz>

Petr Baudis <pasky@suse.cz> writes:

Show 23 quoted lines

> On Fri, Apr 27, 2007 at 05:51:36AM CEST, Junio C Hamano wrote:
>> I always considered it was a bug that the native transport sends
>> SHA-1 of HEAD after dereferencing the symref, instead of saying
>> which branch it points at.
>
> Yes, you are right. That was what I was originally after, but then got
> sidetracked into working on something else and this easy bit was the
> only thing left done, so I just passed it along. :)
>
>> How about proceeding along these lines?
>
> Looks generally good.
>
>>  * We add native protocol extension to let upload-pack to say
>>    what HEAD symref points at, in addition to the SHA-1 HEAD
>>    points at.  Update peek-remote to show this information like
>>    this:
>> 
>> 	->refs/heads/master<TAB>HEAD<LF>
>>         0d5e6c97...<TAB>HEAD<LF>
>> 	...
>
> Yet another obscure syntax? Why not just reusing the actual ref: syntax?

Yup, ref: is fine.  Please make it so, but after 1.5.2 ;-).