From cd82c3c041139c03fceea7e76fb48280b00d73da Mon Sep 17 00:00:00 2001
From: Rob Landley <rob@landley.net>
Date: Thu, 15 Jun 2006 20:07:57 +0000
Subject: Don't do workarounds for broken shells.  Busybox is about to grow a
 shell that we can run our own testsuite under, and I want it to handle the
 full range of strange and peverted behavior you can find in real world shell
 scripts. In the meantime, I'll just say #!/bin/bash in places that may care.

---
 testsuite/testing.sh | 16 ++++++----------
 1 file changed, 6 insertions(+), 10 deletions(-)

diff --git a/testsuite/testing.sh b/testsuite/testing.sh
index 7897c1622..8bd23d803 100755
--- a/testsuite/testing.sh
+++ b/testsuite/testing.sh
@@ -37,7 +37,7 @@ export SKIP=
 
 # Helper functions
 
-optional ()
+optional()
 {
   option=`echo "$OPTIONFLAGS" | egrep "(^|:)$1(:|\$)"`
   # Not set?
@@ -55,7 +55,6 @@ testing ()
 {
   NAME="$1"
   [ -z "$1" ] && NAME=$2
-  ret=0
 
   if [ $# -ne 5 ]
   then
@@ -77,15 +76,12 @@ testing ()
   echo -ne "$5" | eval "$2" > actual
   RETVAL=$?
 
-  cmp expected actual > /dev/null || ret=$?
-  if [ $ret -ne 0 ]
+  cmp expected actual > /dev/null 
+  if [ $? -ne 0 ]
   then
     FAILCOUNT=$[$FAILCOUNT+1]
     echo "FAIL: $NAME"
-    if [ -n "$VERBOSE" ]
-    then
-      diff -u expected actual || /bin/true
-    fi
+    [ -n "$VERBOSE" ] && diff -u expected actual
   else
     echo "PASS: $NAME"
   fi
@@ -101,7 +97,7 @@ testing ()
 # the file is assumed to already be there and only its library dependencies
 # are copied.
 
-mkchroot ()
+function mkchroot
 {
   [ $# -lt 2 ] && return
 
@@ -130,7 +126,7 @@ mkchroot ()
 # Needed commands listed on command line
 # Script fed to stdin.
 
-dochroot ()
+function dochroot
 {
   mkdir tmpdir4chroot
   mount -t ramfs tmpdir4chroot tmpdir4chroot
-- 
cgit v1.2.3