tfendin commented

Feb 25, 2022

•

edited

Loading

This is one way to solve issue #953, but it's a little bit flawed.
The problem is that we used xargs --max-args 1 --replace % -c bash "effort \"%\"", because the function effort only takes one argument and we want it to be quoted. --max-args and --replace are conflicting in newer versions of xargs.
In this PR I've removed xargs and only use bash code to call effort. However, the previous implementation used the --max-procs=4 option to xargs to use effort in four "threads". This version forks heavily, which could cause performance problems in the rest of the system.
I have only done some minor tests in one repo, it should be tested in more repos and on other OS:es as well.

tfendin mentioned this pull request

Closed