diff options
author | Dan McGee <dan@archlinux.org> | 2011-01-12 01:43:28 +0100 |
---|---|---|
committer | Dan McGee <dan@archlinux.org> | 2011-01-12 01:43:28 +0100 |
commit | 0d4dd09993971924d379be4d0944d72f4c77b021 (patch) | |
tree | af6a8cbe2cc2095f0fbccf9c005d8f73de11a3d7 | |
parent | a9cbd15260b396c38b7d5ffe5669994e65f9a86f (diff) | |
download | pacman-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>
-rwxr-xr-x | test/pacman/pmdb.py | 10 |
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): """ |