summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDan McGee <dan@archlinux.org>2011-07-29 18:49:37 -0500
committerDan McGee <dan@archlinux.org>2011-08-02 08:20:27 -0400
commit573260556db49c62f5cf751f53812b7e06c1decb (patch)
tree8f3376371e086f81bcba9fe675aee102b73ea4db
parent66d99957114e98ce41052fcd33200d8fbfbd9f26 (diff)
downloadpacman-573260556db49c62f5cf751f53812b7e06c1decb.tar.xz
pactest: use subprocess module instead of os.system
This is more in line with standard Python practice, and makes keyboard interrupts behave a lot more sanely. It also prevents the useless spawning of a shell as well as simplifies the command building and working directory stuff. Signed-off-by: Dan McGee <dan@archlinux.org>
-rw-r--r--test/pacman/pmtest.py46
1 files changed, 23 insertions, 23 deletions
diff --git a/test/pacman/pmtest.py b/test/pacman/pmtest.py
index e38d3a66..0572d6f0 100644
--- a/test/pacman/pmtest.py
+++ b/test/pacman/pmtest.py
@@ -17,8 +17,10 @@
import os
+import shlex
import shutil
import stat
+import subprocess
import time
import pmrule
@@ -181,7 +183,7 @@ class pmtest(object):
print "==> Running test"
vprint("\tpacman %s" % self.args)
- cmd = [""]
+ cmd = []
if os.geteuid() != 0:
fakeroot = util.which("fakeroot")
if not fakeroot:
@@ -194,41 +196,39 @@ class pmtest(object):
cmd.append("fakechroot")
if pacman["gdb"]:
- cmd.append("libtool execute gdb --args")
+ cmd.extend(["libtool", "execute", "gdb", "--args"])
if pacman["valgrind"]:
- cmd.append("valgrind -q --tool=memcheck --leak-check=full --show-reachable=yes --suppressions=%s/valgrind.supp" % os.getcwd())
- cmd.append("\"%s\" --config=\"%s\" --root=\"%s\" --dbpath=\"%s\" --cachedir=\"%s\"" \
- % (pacman["bin"],
- os.path.join(self.root, util.PACCONF),
- self.root,
- os.path.join(self.root, util.PM_DBPATH),
- os.path.join(self.root, util.PM_CACHEDIR)))
+ cmd.extend(["libtool", "execute", "valgrind", "-q",
+ "--tool=memcheck", "--leak-check=full",
+ "--show-reachable=yes", "--suppressions=%s/valgrind.supp" % os.getcwd()])
+ cmd.extend([pacman["bin"],
+ "--config", os.path.join(self.root, util.PACCONF),
+ "--root", self.root,
+ "--dbpath", os.path.join(self.root, util.PM_DBPATH),
+ "--cachedir", os.path.join(self.root, util.PM_CACHEDIR)])
if not pacman["manual-confirm"]:
cmd.append("--noconfirm")
if pacman["debug"]:
cmd.append("--debug=%s" % pacman["debug"])
- cmd.append("%s" % self.args)
- if not pacman["gdb"] and not pacman["valgrind"] and not pacman["nolog"]:
- cmd.append(">\"%s\" 2>&1" % os.path.join(self.root, util.LOGFILE))
+ cmd.extend(shlex.split(self.args))
+ if not (pacman["gdb"] or pacman["valgrind"] or pacman["nolog"]):
+ output = open(os.path.join(self.root, util.LOGFILE), 'w')
+ else:
+ output = None
vprint("\trunning: %s" % " ".join(cmd))
# Change to the tmp dir before running pacman, so that local package
# archives are made available more easily.
- curdir = os.getcwd()
- tmpdir = os.path.join(self.root, util.TMPDIR)
- os.chdir(tmpdir)
-
time_start = time.time()
- self.retcode = os.system(" ".join(cmd))
+ self.retcode = subprocess.call(cmd, stdout=output, stderr=output,
+ cwd=os.path.join(self.root, util.TMPDIR))
time_end = time.time()
- vprint("\ttime elapsed: %ds" % (time_end - time_start))
+ vprint("\ttime elapsed: %.2fs" % (time_end - time_start))
+
+ if output:
+ output.close()
- if self.retcode == None:
- self.retcode = 0
- else:
- self.retcode /= 256
vprint("\tretcode = %s" % self.retcode)
- os.chdir(curdir)
# Check if the lock is still there
if os.path.isfile(util.PM_LOCK):