summaryrefslogtreecommitdiffstats
path: root/test
diff options
context:
space:
mode:
authorDan McGee <dan@archlinux.org>2011-01-12 01:43:28 +0100
committerDan McGee <dan@archlinux.org>2011-01-12 01:43:28 +0100
commit0d4dd09993971924d379be4d0944d72f4c77b021 (patch)
treeaf6a8cbe2cc2095f0fbccf9c005d8f73de11a3d7 /test
parenta9cbd15260b396c38b7d5ffe5669994e65f9a86f (diff)
downloadpacman-0d4dd09993971924d379be4d0944d72f4c77b021.tar.gz
pacman-0d4dd09993971924d379be4d0944d72f4c77b021.tar.xz
pactest: build the filelist using a set()
This will prevent duplicates, which we had plenty of once I made a few tests that had a list of files greater than the normal two. The previous logic was not working quite right. Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'test')
-rwxr-xr-xtest/pacman/pmdb.py10
1 files changed, 4 insertions, 6 deletions
diff --git a/test/pacman/pmdb.py b/test/pacman/pmdb.py
index 60460ce2..b4d0e2d5 100755
--- a/test/pacman/pmdb.py
+++ b/test/pacman/pmdb.py
@@ -38,16 +38,14 @@ def _mkfilelist(files):
usr/local/bin/
usr/local/bin/dummy
"""
- i = []
+ file_list = set()
for f in files:
dir = getfilename(f)
- i.append(dir)
+ file_list.add(dir)
while "/" in dir:
[dir, tmp] = dir.rsplit("/", 1)
- if not dir + "/" in files:
- i.append(dir + "/")
- i.sort()
- return i
+ file_list.add(dir + "/")
+ return sorted(file_list)
def _mkbackuplist(backup):
"""