Re: [PATCH v2 10/10] receive-pack: convert receive hooks to hook API

From: Adrian Ratiu <adrian.ratiu@collabora.com>
Date: Nov 17, 2025, 16:51 UTC
Message-ID: <87a50kbo1x.fsf@collabora.com>
In-Reply-To: <xmqq346ff56h.fsf@gitster.g>

On Sat, 15 Nov 2025, Junio C Hamano <gitster@pobox.com> wrote:

Show 12 quoted lines

> Adrian Ratiu <adrian.ratiu@collabora.com> writes: 
> 
>> +static int feed_receive_hook(int hook_stdin_fd, struct 
>> receive_hook_feed_state *state, int lines_batch_size) 
> 
> Overly long line and cannot read.  Can you stick to 80-column 
> lines? 
> 
> In any case, the reason I am responding to this message is not 
> about coding styles, but it seems to be the one whose leak is 
> holding the CI job from passing at the tip of 'seen'. 
>

Hi Junio and thanks for these; will address and send a refresh with the rest of feedback in v3 very soon.

$ 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.