From: Shawn O. Pearce Date: Tue, 14 Oct 2008 15:02:27 GMT Subject: Re: [PATCH v2] Fix fetch/pull when run without --update-head-ok Message-ID: <20081014150227.GC4856@spearce.org> In-Reply-To: Johannes Schindelin wrote: > On Mon, 13 Oct 2008, Daniel Barkalow wrote: > > > On Mon, 13 Oct 2008, Johannes Schindelin wrote: > > > > > I actually understand now why the tests started failing: the change from > > > resolve_ref() to get_branch() as requested by Daniel are at fault: > > > get_branch() does not check if the branch has an initial commit. > > So, my vote is to revert back to resolve_ref(), even if it needs more > lines. Yes, I agree, resolve_ref() is the best thing to be using here, even if it is more code. get_branch() validates the commit and we don't want that. We really just want to know if the current branch is going to be updated, we don't care to what/why. -- Shawn.