Re: [PATCH 1/6] run-command API: replace run_processes_parallel_tr2() with opts struct

From: Junio C Hamano <gitster@pobox.com>
Date: Apr 29, 2022, 16:44 UTC
Message-ID: <xmqqv8urdekz.fsf@gitster.g>
In-Reply-To: <YmsgWj5vPEWNyGFA@google.com>

Emily Shaffer <emilyshaffer@google.com> writes:

Show 7 quoted lines

> If I'm being honest, I'd rather see everything _but_ the trace2 stuff go
> into an opts struct, and then see the same entry points we have today
> (run_processes_parallel that takes a struct, run_processes_parallel_tr2
> that takes a struct and two tr2 string args). Or, I guess, a single
> run_processes_parallel() that only takes a struct, does the right thing
> with the trace args, and entirely removes the
> run_processes_parallel_tr2 call.

Yup, it was the impression I had when I saw this patch for the first time.

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