Re: [PATCH 0/5] run-command API: get rid of "argv"

From: Junio C Hamano <gitster@pobox.com>
Date: Nov 22, 2021, 18:11 UTC
Message-ID: <xmqq7dd0giwp.fsf@gitster.g>
In-Reply-To: <YZvY+BJhxaFIOdnJ@coredump.intra.peff.net>

Jeff King <peff@peff.net> writes:

> I'm on the fence on how important it is to do these cleanups. IMHO they
> are half of what really sells the change in the first place (since the
> other bug can pretty easily be fixed without it).

Yes. I actually think these have much better value for their complexity, compared to the other "half" of the topic ;-)

> But maybe it is piling too much onto what is already a pretty big
> change. The cleanups could be done individually later.

I am OK with that, too. But I do agree that the series is too big to sit in front of a fix for a bug, for which a much simpler and focused approach has already been discussed, to block it.

Thanks.

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