diff options
| author | Michael Tokarev | 2013-11-10 22:01:38 +0100 |
|---|---|---|
| committer | Bernhard Reutner-Fischer | 2013-11-10 22:06:06 +0100 |
| commit | afa63b2dcdc9b9d0183ffd84599ea5d4ad94d639 (patch) | |
| tree | af1b15bd42d42ee5be891983d86532e6e41c7ab8 | |
| parent | 5e63776212f8fab4fba08ab2b99dd5241f0c3880 (diff) | |
| download | busybox-afa63b2dcdc9b9d0183ffd84599ea5d4ad94d639.zip busybox-afa63b2dcdc9b9d0183ffd84599ea5d4ad94d639.tar.gz | |
fix bashisms in testsuite/
These places use ${var/search/replace}, $((n++) and
`type -p' constructs which are not-so-standard.
Replace with equivalent constructs.
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
| -rw-r--r-- | testsuite/hostid/hostid-works | 10 | ||||
| -rwxr-xr-x | testsuite/md5sum.tests | 6 | ||||
| -rw-r--r-- | testsuite/testing.sh | 8 | ||||
| -rw-r--r-- | testsuite/which/which-uses-default-path | 2 |
4 files changed, 12 insertions, 14 deletions
diff --git a/testsuite/hostid/hostid-works b/testsuite/hostid/hostid-works index bcfd717..8c20bdf 100644 --- a/testsuite/hostid/hostid-works +++ b/testsuite/hostid/hostid-works @@ -1,8 +1,6 @@ h=x$(busybox hostid) # Is $h a sequence of hex numbers? -x="${h//[0123456789abcdef]/x}" -x="${x//xxx/x}" -x="${x//xxx/x}" -x="${x//xxx/x}" -x="${x//xx/x}" -test x"$x" = x"x" +case "$h" in + x*[!0-9a-f]*) false;; + *) true;; +esac diff --git a/testsuite/md5sum.tests b/testsuite/md5sum.tests index 1068b08..6c75b6d 100755 --- a/testsuite/md5sum.tests +++ b/testsuite/md5sum.tests @@ -30,13 +30,13 @@ result=`( n=0 while test $n -le 999; do echo "$text" | head -c $n | "$sum" - : $((n++)) + n=$(($n+1)) done | "$sum" )` if test x"$result" = x"$expected -"; then - echo "PASS: $sum" - exit 0 + echo "PASS: $sum" + exit 0 fi echo "FAIL: $sum (r:$result exp:$expected)" diff --git a/testsuite/testing.sh b/testsuite/testing.sh index e7e64e5..f5b7569 100644 --- a/testsuite/testing.sh +++ b/testsuite/testing.sh @@ -56,10 +56,10 @@ optional() { SKIP= while test "$1"; do - if test x"${OPTIONFLAGS/*:$1:*/y}" != x"y"; then - SKIP=1 - return - fi + case "${OPTIONFLAGS}" in + *:$1:*) ;; + *) SKIP=1; return ;; + esac shift done } diff --git a/testsuite/which/which-uses-default-path b/testsuite/which/which-uses-default-path index 63ceb9f..349583d 100644 --- a/testsuite/which/which-uses-default-path +++ b/testsuite/which/which-uses-default-path @@ -1,4 +1,4 @@ -BUSYBOX=$(type -p busybox) +BUSYBOX=$(command -pv busybox) SAVED_PATH=$PATH unset PATH $BUSYBOX which ls |