pactest: only use fakeroot when it is found.

Only use fakeroot and fakechroot when they are found AND required.
fakechroot only had the first condition, and fakeroot only the second.

When they are required (user != root) but not found, display a warning.

Signed-off-by: Xavier Chantry <shiningxc@gmail.com>
Signed-off-by: Dan McGee <dan@archlinux.org>

1 files changed, 10 insertions, 6 deletions

diff --git a/pactest/pmtest.py b/pactest/pmtest.py
index 267eeb2e..e8f6fa8d 100755
--- a/pactest/pmtest.py
+++ b/pactest/pmtest.py
@@ -189,13 +189,17 @@ class pmtest:
 
         cmd = [""]
         if os.geteuid() != 0:
-            cmd.append("fakeroot")
+            fakeroot = which("fakeroot")
+            if not fakeroot:
+                print "WARNING: fakeroot not found!"
+            else:
+                cmd.append("fakeroot")
 
-        fakechroot = which("fakechroot")
-        if not fakechroot:
-            print "WARNING: fakechroot not found, scriptlet tests WILL fail!!!"
-        else:
-            cmd.append("fakechroot")
+            fakechroot = which("fakechroot")
+            if not fakechroot:
+                print "WARNING: fakechroot not found, scriptlet tests WILL fail!!!"
+            else:
+                cmd.append("fakechroot")
 
         if pacman["gdb"]:
             cmd.append("libtool execute gdb --args")