diff options
| author | Dan McGee <dan@archlinux.org> | 2007-02-28 17:37:24 +0100 |
|---|---|---|
| committer | Dan McGee <dan@archlinux.org> | 2007-02-28 17:37:24 +0100 |
| commit | 13e21110459aaf99dd739802c2b07b3d5b9e2a68 (patch) | |
| tree | e8df3fa922cbd076185e56f1cd0e3a665c17b58d /pactest/util.py | |
| parent | bdac9105892dae8b74952f86300e6161363b93d4 (diff) | |
| download | pacman-13e21110459aaf99dd739802c2b07b3d5b9e2a68.tar.gz pacman-13e21110459aaf99dd739802c2b07b3d5b9e2a68.tar.xz | |
* Unifying placement of REPLACES in desc file, as pacman2 does. We'll worry
about bigger DB changes later, but lets not screw anything up for release.
* Removed some weird uses of "not ... ==" usage in pactest- correct me if I'm
wrong, but isn't "!=" a lot more clean and concise?
* Print description of failed tests in the pactest summary. This could get
dirty with a lot of failed tests though, so watch out.
Diffstat (limited to 'pactest/util.py')
| -rwxr-xr-x | pactest/util.py | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/pactest/util.py b/pactest/util.py index a633e869..2b384098 100755 --- a/pactest/util.py +++ b/pactest/util.py @@ -62,7 +62,7 @@ def getfilename(name): """ filename = "" link = "" - if not name.find(" -> ") == -1: + if name.find(" -> ") != -1: filename, link = name.split(" -> ") elif name[-1] == "*": filename = name.rstrip("*") @@ -80,7 +80,7 @@ def mkfile(name, data = ""): link = "" filename = "" - if not name.find(" -> ") == -1: + if name.find(" -> ") != -1: islink = 1 filename, link = name.split(" -> ") elif name[-1] == "*": @@ -181,8 +181,9 @@ def mkcfgfile(filename, root, option, db): # Repositories data.extend(["[%s]\n" \ "server = file://%s\n" \ - % (value.treename, os.path.join(root, SYNCREPO, value.treename)) \ - for key, value in db.iteritems() if not key == "local"]) + % (value.treename, \ + os.path.join(root, SYNCREPO, value.treename)) \ + for key, value in db.iteritems() if key != "local"]) mkfile(os.path.join(root, filename), "\n".join(data)) |