testsuite/runtest: remove (last) bashism
authorDenis Vlasenko <vda.linux@googlemail.com>
Fri, 25 Apr 2008 08:43:01 +0000 (08:43 -0000)
committerDenis Vlasenko <vda.linux@googlemail.com>
Fri, 25 Apr 2008 08:43:01 +0000 (08:43 -0000)
mkswap: fix selinux build

testsuite/runtest
util-linux/mkswap.c

index c668240eb245556d946683335b5f60a5ffe220f9..478764801a30dc93c458e33c2dd9a99f56a4e528 100755 (executable)
@@ -9,12 +9,12 @@
 # Option -e will make testcase stop on the first failed command.
 run_applet_testcase()
 {
-       local applet=$1
+       local applet="$1"
        local testcase="$2"
 
        local status
        local uc_applet=$(echo $applet | tr a-z A-Z)
-       local testname=`basename "$testcase"`
+       local testname=$(basename "$testcase")
 
        if grep -q "^# CONFIG_${uc_applet} is not set$" $bindir/.config; then
                echo "UNTESTED: $testname"
@@ -22,7 +22,7 @@ run_applet_testcase()
        fi
 
        if grep -q "^# FEATURE: " "$testcase"; then
-               local feature=`sed -ne 's/^# FEATURE: //p' "$testcase"`
+               local feature=$(sed -ne 's/^# FEATURE: //p' "$testcase")
 
                if grep -q "^# ${feature} is not set$" $bindir/.config; then
                        echo "UNTESTED: $testname"
@@ -58,7 +58,7 @@ run_applet_tests()
        local applet=$1
        local status=0
        for testcase in $tsdir/$applet/*; do
-               case `basename "$testcase"` in
+               case $(basename "$testcase") in
                        \#*)
                                continue
                                ;;
@@ -130,9 +130,12 @@ for applet in $applets; do
 
        # Is this a new-style test?
        if [ -f "${applet}.tests" ]; then
-               if [ ! -h "$LINKSDIR/$applet" ] && [ "${applet:0:4}" != "all_" ]; then
-                       echo "SKIPPED: $applet (not built)"
-                       continue
+               if [ ! -h "$LINKSDIR/$applet" ]; then
+                       # (avoiding bash'ism "${applet:0:4}")
+                       if ! echo "$applet" | grep "^all_" >/dev/null; then
+                               echo "SKIPPED: $applet (not built)"
+                               continue
+                       fi
                fi
 #              echo "Running test ${tsdir:-.}/${applet}.tests"
                PATH="$LINKSDIR:$tsdir:$bindir:$PATH" "${tsdir:-.}/${applet}.tests"
index 1c364efe9f2dec3bd690f10eda2042502a2115ef..11c411b6ac14043f3ce2f0bfcb83b740c235638d 100644 (file)
@@ -23,7 +23,7 @@ static void mkswap_selinux_setcontext(int fd, const char *path)
                security_context_t oldcon = NULL;
                context_t context;
 
-               if (fgetfilecon_raw(fd, &oldcon) < 0) {
+               if (fgetfilecon(fd, &oldcon) < 0) {
                        if (errno != ENODATA)
                                goto error;
                        if (matchpathcon(path, stbuf.st_mode, &oldcon) < 0)