On 10/08/18 00:04, Zac Medico wrote:
> For USERLAND=BSD, set XARGS="gxargs -r" if gxargs is available,
> so the code from bug 630292 works for USERLAND=BSD.
>
> Fixes: 50283f1abb77 (install-qa-check.d/60pngfix: parallel support (bug 
> 630292))
> Reported-by: Michał Górny <mgo...@gentoo.org>
> Bug: https://bugs.gentoo.org/663256
> ---
>  bin/isolated-functions.sh | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/bin/isolated-functions.sh b/bin/isolated-functions.sh
> index 28ca94532..cac42a4c5 100644
> --- a/bin/isolated-functions.sh
> +++ b/bin/isolated-functions.sh
> @@ -448,7 +448,11 @@ fi
>  if [[ -z ${XARGS} ]] ; then
>       case ${USERLAND} in
>       BSD)
> -             export XARGS="xargs"
> +             if type -P gxargs > /dev/null; then
> +                     export XARGS="gxargs -r"
> +             else
> +                     export XARGS="xargs"
> +             fi
>               ;;
>       *)
>               export XARGS="xargs -r"
Isn't the else clause there redundant?

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to