[PATCH] Make MAKEALL a bash script

Jean-Christophe PLAGNIOL-VILLARD plagnioj at jcrosoft.com
Mon May 2 11:44:10 EDT 2011


On 11:50 Mon 02 May     , Uwe Kleine-König wrote:
> MAKEALL uses the array PIPESTATUS which is a bash extension. A
> replacement isn't available in other common shells (e.g. dash which
> provides /bin/sh in Debian since at least Squeeze (6.0))
> 
> While at it, use . instead of source which is another bashism. Not
> necessary any more, but there is no need to use a bash extension if
> there is a generic way, too.
> 
> Reported-by: Belisko Marek <marek.belisko at gmail.com>
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig at pengutronix.de>
Acked-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj at jcrosoft.com>

Dash have no capability equivalient as PIPESTATUS

when I write the script I try to do not do bashism but here no choice

Best Regards,
J.



More information about the barebox mailing list