From: Kang-Che Sung <explorer09@gmail.com>
Date: Sat, 15 Jul 2017 03:38:58 +0000 (+0800)
Subject: make_single_applets: fix ": $((fail++))" expansion error
X-Git-Tag: 1_27_1~2
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=6f13dd1d6574bbc0eacd5574d2e451596a92df5d;p=oweals%2Fbusybox.git

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>
---

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