Re: [PATCH] coccicheck: optionally batch spatch invocations

From

Junio C Hamano <gitster@pobox.com>

Date

May 7, 2019, 02:42 UTC

Message-ID

<xmqq7eb3nfxn.fsf@gitster-ct.c.googlers.com>

In-Reply-To

<20190506234334.GA13296@sigill.intra.peff.net>

Jeff King <peff@peff.net> writes:

> The implementation is done with xargs, which should be widely available;
> it's in POSIX, we rely on it already in the test suite. And "coccicheck"
> is really a developer-only tool anyway, so it's not a big deal if
> obscure systems can't run it.

OK.

> I left the default at 1 for safety. Probably 4 or 16 would be an OK
> default, but I don't have any interest in figuring out exactly what
> Travis or some hypothetical average machine can handle. I'll be setting
> mine to 999. ;)
>
> Making "0" work as "unlimited" might be nice, but xargs doesn't support
> that and I didn't want to make the recipe any more unreadable than it
> already is.

Thanks.