Re: 6d4bb3833c3d2114d (fetch: verify we have everything we need before updating our ref) breaks fetch
- From
Philip Oakley <philipoakley@iee.org>
- Date
- Oct 1, 2011, 10:38 UTC
- Message-ID
- <BED2B875AEDF4E52843F1C9211A8D2A1@PhilipOakley>
- In-Reply-To
- <20111001060353.GA25228@sigill.intra.peff.net>
From: "Jeff King" <peff@peff.net>
Show 22 quoted lines
> On Sat, Oct 01, 2011 at 01:54:08AM +0200, Carlos Martín Nieto wrote: > >> > Have you tried >> > >> > $ ./git fetch git://repo.or.cz/git >> >> But this would execute /usr/local/libexec/git-fetch, wouldn't it? That >> is precisely what I don't want to execute, because I changed some code >> in builtin/fetch.c that I want to test. > > No, but only because fetch is a builtin. However, it still doesn't set > up exec_path correctly, so your rev-list problem would not go away. > >> I guess I'll have to either properly install git from 'next' or base my >> changed on 'maint' > > Just use bin-wrappers/git. That's exactly what it's there for (and it's > what the test scripts use to make sure we are testing what is compiled). > > Your change isn't the problem; only your testing method. > > -Peff
Peff,
Is there a write up of the the git testing method and how to use bin-wrappers etc. I didn't see anything in the Documentation, but I may not have looked carefully enough
Philip