* 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.

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))