commit 1a7bbd255fd02fbfee6a26b990d15d38402f1992
author Steve Baker <sbaker@redhat.com> Mon Dec 03 17:04:02 2012 +1300
committer Steve Baker <sbaker@redhat.com> Mon Dec 03 17:04:02 2012 +1300
tree a492085bcb1de5a915e3e43fa4b3d7dc5b34785b
parent 6540d8910194bb523601ffdd06cdf4c2126e3fd0

is_suse false positives on Fedora

Also, uses of is_suse were also always evaluating to true
on Fedora.

Change-Id: I068f3179edbfb295163a4e4faa4998f2f7b2c124

functions

diff --git a/functions b/functions
index f2b12e2..9022675 100644
--- a/functions
+++ b/functions
@@ -349,8 +349,7 @@
         GetOSVersion
     fi
 
-    [[ "$os_VENDOR" = "openSUSE" || "$os_VENDOR" = "SUSE LINUX" ]]
-    return $?
+    [ "$os_VENDOR" = "openSUSE" ] || [ "$os_VENDOR" = "SUSE LINUX" ]
 }
 
 
@@ -646,7 +645,7 @@
         SUDO_PIP="env"
     else
         SUDO_PIP="sudo"
-        if [[ "$os_PACKAGE" = "deb" || is_suse ]]; then
+        if [[ "$os_PACKAGE" = "deb" ]] || is_suse; then
             CMD_PIP=/usr/bin/pip
         else
             CMD_PIP=/usr/bin/pip-python
@@ -1040,7 +1039,7 @@
         GetOSVersion
     fi
 
-    if [[ "$os_PACKAGE" = "deb" || is_suse ]]; then
+    if [[ "$os_PACKAGE" = "deb" ]] || is_suse; then
         echo "/usr/local/bin/$module-rootwrap"
     else
         echo "/usr/bin/$module-rootwrap"
@@ -1057,7 +1056,7 @@
 
     # Qpid was introduced to Ubuntu in precise, disallow it on oneiric; it is
     # not in openSUSE either right now.
-    [[ "$DISTRO" = "oneiric" || is_suse ]]
+    [[ "$DISTRO" = "oneiric" ]] || is_suse
     return $?
 }