From: Junio C Hamano Date: Thu, 23 Apr 2009 08:36:58 GMT Subject: Re: [PATCH] tests: remove xargs in favor of --stdin where possible Message-ID: <7vfxfzwytx.fsf@gitster.siamese.dyndns.org> In-Reply-To: <1240468281-23815-1-git-send-email-pclouds@gmail.com> Nguyễn Thái Ngọc Duy writes: > Signed-off-by: Nguyễn Thái Ngọc Duy > --- > Patch "Convert to use quiet option when available" reminds me I had a similar > patch to remove (mostly) xargs usage from tests. The reason being...? (1) xargs is not universally available, or portable enough? (2) people may learn from tests, and we should demostrate our ability? (3) something else? If the reason is (1) I would understand it, but if it is (2) then I am a bit reluctant. The tests may not break with your change because none of them may use problematic characters (especially "\n" and '"'), but update-index --stdin without -z is not suitable for reading from output from "find" without -0 option (on the other hand, "update-index -z --stdin" is good for reading output from "find -0"; but for portability we avoid GNUism "find -0").