Re: [PATCH 1/4] run-command: add pipe_output to run_processes_parallel

From: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date: Sep 27, 2022, 19:34 UTC
Message-ID: <220927.86fsgcy5j9.gmgdl@evledraar.gmail.com>
In-Reply-To: <CAFySSZBSAW=zea8UoMiaQkf8rdJUBGHDYZQFkPLW7mRSciS-FA@mail.gmail.com>

On Tue, Sep 27 2022, Calvin Wan wrote:

Show 11 quoted lines

>> I don't know if you started this work or not, but I had a WIP rebase of
>> those on-list patches lying around in my tree, which I'm rebasing on
>> "master" currently.
>>
>> Some of it's a bit tricky with the in-tree activity since then, I'll
>> send them in when they're ready, maybe you already did your own rebasing
>> of them, or maybe it helps & you'd like to base a re-submission on top
>> of them.
>
> I have not started on this part yet. Do you have an estimate as to when
> you're planning on submitting your rebase?

I hacked this up today, it passes CI at least:

	https://github.com/git/git/compare/master...avar:avar/hook-run-process-parallel-tty-regression-2-argument-passing

> I'm also considering not using
> my pipe_output option and going a different route since there is the
> issue of dealing with potentially unbounded output.

Okey, if it's not blocking a re-submission of yours then I'll definitely wait until after the RC to submit the above, at least, but if you'd like it earlier...