summaryrefslogtreecommitdiffstats
path: root/pactest/pmdb.py
diff options
context:
space:
mode:
authorChantry Xavier <shiningxc@gmail.com>2008-04-17 09:02:11 +0200
committerDan McGee <dan@archlinux.org>2008-04-17 21:29:08 +0200
commitc465d9e848b19b495259c7021a583c29fba92b44 (patch)
tree3860d73280a51a812f9625f5269f4e97b8093240 /pactest/pmdb.py
parent5e375aa9d387b2ef703dd35f60df1daad6a40237 (diff)
downloadpacman-c465d9e848b19b495259c7021a583c29fba92b44.tar.gz
pacman-c465d9e848b19b495259c7021a583c29fba92b44.tar.xz
pactest : Use tarfile module.
Previously, tar was called manually with os.system. This caused one fork per package/db creation, which is costly, especially on cygwin. Besides, it also caused some problems with directory with whitespaces (that could also be fixed with quotes, but well..) Using tarfile module is cleaner and more efficient, and still easy enough. Benchmark (time make check) : - windows / cygwin prepatch: real 6m36.360s user 2m28.914s sys 2m35.866s postpatch: real 5m25.428s user 1m26.029s sys 2m0.006s - linux prepatch: real 1m22.629s user 0m31.498s sys 0m18.899s postpatch: real 1m11.465s user 0m26.382s sys 0m12.986s Signed-off-by: Chantry Xavier <shiningxc@gmail.com>
Diffstat (limited to 'pactest/pmdb.py')
-rwxr-xr-xpactest/pmdb.py9
1 files changed, 8 insertions, 1 deletions
diff --git a/pactest/pmdb.py b/pactest/pmdb.py
index e0f328ef..cfa146bd 100755
--- a/pactest/pmdb.py
+++ b/pactest/pmdb.py
@@ -19,6 +19,7 @@
import os
import tempfile
import shutil
+import tarfile
import pmpkg
from util import *
@@ -343,7 +344,13 @@ class pmdb:
# Generate database archive
mkdir(path)
archive = os.path.join(path, "%s%s" % (self.treename, PM_EXT_DB))
- os.system("tar zcf %s *" % archive)
+ tar = tarfile.open(archive, "w:gz")
+ for root, dirs, files in os.walk('.'):
+ for d in dirs:
+ tar.add(os.path.join(root, d), recursive=False)
+ for f in files:
+ tar.add(os.path.join(root, f))
+ tar.close()
os.chdir(curdir)
shutil.rmtree(tmpdir)