use 'test -n "..."' instead of 'test "..."'; when "..." expands to '-v'
[util-vserver.git] / util-vserver / scripts / vsomething
index a2d497b..46a4b26 100755 (executable)
@@ -95,13 +95,13 @@ res=255
 for i in "${vservers[@]}"; do
     cnt=$[ cnt + 1 ]
 
-    test "$is_quiet" || {
+    test -n "$is_quiet" || {
        colorize bold echo -n "$title: operating on vserver "
        colorize bold colorize emph echo "$i"
        xtermTitle "$title: operating on vserver '$i'  [$cnt/${#vservers[@]}]"
     }
 
-    if test "$VSOMETHING_PKGMGMT"; then
+    if test -n "$VSOMETHING_PKGMGMT"; then
        if pkgmgmt.isInternal "$i"; then
            $_VSERVER "$i" exec "$cmd" "$@"
        else
@@ -114,12 +114,12 @@ for i in "${vservers[@]}"; do
        
     res=$?
 
-    test $res -eq 0 -o "$is_quiet" || {
+    test $res -eq 0 -o -n "$is_quiet" || {
        colorize error echo -n $"$title failed on vserver '$i' with errorcode $res"
        echo
     }
        
-    test "$is_quiet" || echo
+    test -n "$is_quiet" || echo
 done
 
 test "$cnt" -ge 0 || warning $"No vservers specified"