diff options
author | Dan McGee <dan@archlinux.org> | 2011-01-12 01:43:28 +0100 |
---|---|---|
committer | Dan McGee <dan@archlinux.org> | 2011-01-12 16:11:10 +0100 |
commit | cae2bdafec93381284ba487010738d72d0992aab (patch) | |
tree | 20a9aa26eea63988463671ce50e44313b65e41e8 | |
parent | 8e30a46adbe350a9741bbb9748334a4adc9742f6 (diff) | |
download | pacman-cae2bdafec93381284ba487010738d72d0992aab.tar.gz pacman-cae2bdafec93381284ba487010738d72d0992aab.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>
(cherry picked from commit 0d4dd09993971924d379be4d0944d72f4c77b021)
-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 8cb1b832..218e3b58 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): """ |