From: Junio C Hamano <junkio@cox.net>
Date: Fri, 16 Feb 2007 07:31:51 GMT
Subject: Re: [PATCH] Add git-unbundle - unpack objects and references for disconnected transfer
Message-ID: <7v4ppma788.fsf@assigned-by-dhcp.cox.net>
In-Reply-To: <20070216064852.GF28894@spearce.org>

"Shawn O. Pearce" <spearce@spearce.org> writes:

> Mark Levedahl <mdl123@verizon.net> wrote:
>> +        # update only if non-fastforward
>> +        local=$(git-rev-parse --verify "$ref^0" 2>/dev/null)
>> ...
>> +        git-update-ref -m "git-unbundle update" $ref $sha1
>
> What about passing $local as the final argument to update-ref,
> so that the ref won't be modified if someone changed it while
> an unbundle was running?  Sure its mostly a manual operation,
> but imagine running it on a bare repository while someone else
> is pushing into it...

The script already has $local at that point, so adding it to
update-ref is a no-cost change to make things safer.  I think it
makes sense.

But I have to wonder...  While someone else is _pushing_ into
it?  Why are _you_ sneakernetting, then?