diff options
| author | Kang-Che Sung <explorer09@gmail.com> | 2017-07-15 11:38:58 +0800 |
|---|---|---|
| committer | Denys Vlasenko <vda.linux@googlemail.com> | 2017-07-15 19:20:45 +0200 |
| commit | ed2b9225371a36dd2694df65e395dc32fd94c5dd (patch) | |
| tree | 55ec9e9e50b63aba14fa22c1524391e55d977d96 | |
| parent | c4ddf04b6898c218b08251d81d30c3a2d587b10c (diff) | |
| download | busybox-ed2b9225371a36dd2694df65e395dc32fd94c5dd.tar.gz | |
make_single_applets: fix ": $((fail++))" expansion error
$((fail++)) is not a required expression in POSIX, and in "dash" it
could produce an error like this:
./make_single_applets.sh: 61: arithmetic expression: expecting primary: "fail++"
Replace this with something portable: fail=$((fail+1)) would work.
Signed-off-by: Kang-Che Sung <explorer09@gmail.com>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
| -rwxr-xr-x | make_single_applets.sh | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/make_single_applets.sh b/make_single_applets.sh index 8ad7a7406..329a27d32 100755 --- a/make_single_applets.sh +++ b/make_single_applets.sh @@ -54,18 +54,18 @@ for app; do fi if ! yes '' | make oldconfig >busybox_make_${app}.log 2>&1; then - : $((fail++)) + fail=$((fail+1)) echo "Config error for ${app}" mv .config busybox_config_${app} elif ! make $makeopts >>busybox_make_${app}.log 2>&1; then - : $((fail++)) + fail=$((fail+1)) grep -i -e error: -e warning: busybox_make_${app}.log echo "Build error for ${app}" mv .config busybox_config_${app} elif ! grep -q '^#define NUM_APPLETS 1$' include/NUM_APPLETS.h; then grep -i -e error: -e warning: busybox_make_${app}.log mv busybox busybox_${app} - : $((fail++)) + fail=$((fail+1)) echo "NUM_APPLETS != 1 for ${app}: `cat include/NUM_APPLETS.h`" mv .config busybox_config_${app} else |