On Tue, May 24, 2011 at 10:11:33AM -0400, Joey Hess wrote:
>> tr \\n \\0 < movelist | xargs -0 -i rm -f '{}'
>> A separate pair of shell and `rm' processes gets spawned per
>> individual file to be removed. The `tr' is redundant, but
>> insignificant.
>> - complex_doit("(cd $sourcedir >/dev/null ; tr '\\n' '\\0' <
>> $pwd/debian/movelist | xargs -0 -i rm -f '{}')");
>> + complex_doit("(cd $sourcedir >/dev/null ; <
>> $pwd/debian/movelist xargs -rd'\\n' rm -f)");
> This is not valid shell code. -----------------------------^^^
Well, I, my shells, their man pages and other documentation
don't agree. And I was actually running these modifications to
test and time them. But I don't mean to insist.
> All that is really needed to speed this up is removal of the
> -i and '{}' options from xargs.
Sure thing, that's the crux of the fix. Please make that happen.
--
/Dennis Vshivkov <wal...@amur.ru>
...В гневе желать кому-то смерти -- это болезнь...
--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
