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

Re: [PATCH] fetch-pack: trace packfile URI downloads

From
Junio C Hamano <gitster@pobox.com>
Date
Sep 8, 2026, 03:27 UTC
Message-ID
<xmqqse3kzam2.fsf@gitster.g>
In-Reply-To
<20260907212454.80627-1-tnyman@openai.com>
Ted Nyman <tnyman@openai.com> writes:
Show 11 quoted lines
> On Tue, Aug 11, 2026 at 03:31:44PM +0200, Patrick Steinhardt wrote:
>> Anyway, having this in a separate test would've been nice, but that
>> doesn't warrant a reroll in my eyes. So overall, this patch looks good
>> to me, thanks!
>
> Thanks for reviewing, Patrick.
>
> Junio, a gentle ping on this patch. Patrick was positive on it and didn't
> think the test organization warranted a reroll. Is there anything else
> you'd like me to address before picking it up? I'm happy to split out the
> tracing checks into a separate test or refresh the patch if that helps.

Thanks. This slipped below my radar. It does not look to me that the placement of the test is too bad.

Will queue.
Previous: Ted Nyman
Message 4 of 4 in “fetch-pack: trace packfile URI downloads”
  1. fetch-pack: trace packfile URI downloadsTed Nyman, Jul 26, 2026
  2. Patrick SteinhardtAug 11, 2026
  3. Ted NymanSep 7, 2026
  4. Junio C HamanoSep 8, 2026

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.