Re: [PATCH 6/8] run-command: add an asynchronous parallel child processor

2015-12-14 Thread Stefan Beller
On Mon, Dec 14, 2015 at 12:39 PM, Johannes Sixt wrote: > > I can't quite parse the first sentence in this paragraph. Perhaps something > like this: > > To detect when a child has finished executing, we check interleaved > with other actions (such as checking the liveliness of

[PATCH 6/8] run-command: add an asynchronous parallel child processor

2015-12-14 Thread Stefan Beller
This allows to run external commands in parallel with ordered output on stderr. If we run external commands in parallel we cannot pipe the output directly to the our stdout/err as it would mix up. So each process's output will flow through a pipe, which we buffer. One subprocess can be directly

Re: [PATCH 6/8] run-command: add an asynchronous parallel child processor

2015-12-14 Thread Johannes Sixt
Am 14.12.2015 um 20:37 schrieb Stefan Beller: This allows to run external commands in parallel with ordered output on stderr. If we run external commands in parallel we cannot pipe the output directly to the our stdout/err as it would mix up. So each process's output will flow through a pipe,

Re: [PATCH 6/8] run-command: add an asynchronous parallel child processor

2015-09-30 Thread Junio C Hamano
Junio C Hamano writes: > I may have comments on other parts of this patch, but I noticed this > a bit hard to read while reading the end result. > ... I finished reading the remainder. Other than the above all look sensible. Will replace what had been queued. Thanks. -- To

Re: [PATCH 6/8] run-command: add an asynchronous parallel child processor

2015-09-30 Thread Stefan Beller
On Tue, Sep 29, 2015 at 8:12 PM, Junio C Hamano wrote: > Stefan Beller writes: > >> + while (1) { >> + int i; >> + int output_timeout = 100; >> + int spawn_cap = 4; >> + >> + if (!no_more_task) { >> +

Re: [PATCH 6/8] run-command: add an asynchronous parallel child processor

2015-09-29 Thread Junio C Hamano
Stefan Beller writes: > + while (1) { > + int i; > + int output_timeout = 100; > + int spawn_cap = 4; > + > + if (!no_more_task) { > + for (i = 0; i < spawn_cap; i++) { > + int

[PATCH 6/8] run-command: add an asynchronous parallel child processor

2015-09-28 Thread Stefan Beller
This allows to run external commands in parallel with ordered output on stderr. If we run external commands in parallel we cannot pipe the output directly to the our stdout/err as it would mix up. So each process's output will flow through a pipe, which we buffer. One subprocess can be directly