git/list[1] front-page[2] threads[3] people[4] search[5] about
 

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 
Previous: Jeff KingNext: Jeff King
Message 8 of 9 in “6d4bb3833c3d2114d (fetch: verify we have everything we need before updating our ref) breaks fetch”
  1. Carlos Martín NietoSep 28, 2011
  2. Junio C HamanoSep 28, 2011
  3. Jeff KingSep 28, 2011
  4. Carlos Martín NietoSep 30, 2011
  5. Jakub NarebskiSep 28, 2011
  6. Carlos Martín NietoSep 30, 2011
  7. Jeff KingOct 1, 2011
  8. Philip OakleyOct 1, 2011
  9. Jeff KingOct 1, 2011

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.