From 3829896a8adc371eef76c6af3afb675792e212d6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Daniel=20Mart=C3=AD?= <mvdan@mvdan.cc>
Date: Fri, 24 Oct 2014 22:23:58 +0200
Subject: [PATCH] Fix all pep8 warnings

---
 fdroidserver/common.py  |  5 +----
 fdroidserver/publish.py | 10 +++++-----
 fdroidserver/verify.py  |  6 +++---
 3 files changed, 9 insertions(+), 12 deletions(-)

diff --git a/fdroidserver/common.py b/fdroidserver/common.py
index 9af1779a..033cba8b 100644
--- a/fdroidserver/common.py
+++ b/fdroidserver/common.py
@@ -1812,13 +1812,10 @@ def compare_apks(apk1, apk2, tmp_dir):
         return("Failed to unpack " + apk2)
 
     p = FDroidPopen(['diff', '-r', 'this_apk', 'that_apk'], cwd=tmp_dir,
-            output=False)
+                    output=False)
     lines = p.output.splitlines()
     if len(lines) != 1 or 'META-INF' not in lines[0]:
         return("Unexpected diff output - " + p.output)
 
     # If we get here, it seems like they're the same!
     return None
-
-
-
diff --git a/fdroidserver/publish.py b/fdroidserver/publish.py
index 520a72a0..346a534e 100644
--- a/fdroidserver/publish.py
+++ b/fdroidserver/publish.py
@@ -113,9 +113,9 @@ def main():
 
         # There ought to be valid metadata for this app, otherwise why are we
         # trying to publish it?
-        if not appid in allapps:
+        if appid not in allapps:
             logging.error("Unexpected {0} found in unsigned directory"
-                    .format(apkfilename))
+                          .format(apkfilename))
             sys.exit(1)
         app = allapps[appid]
 
@@ -133,7 +133,7 @@ def main():
                     break
             if not versionname:
                 logging.error("...no defined build for version code {0}"
-                        .format(vercode))
+                              .format(vercode))
                 continue
 
             # Figure out where the developer's binary is supposed to come from...
@@ -147,14 +147,14 @@ def main():
             p = FDroidPopen(['wget', '-nv', url], cwd=tmp_dir)
             if p.returncode != 0 or not os.path.exists(srcapk):
                 logging.error("...failed to retrieve " + url +
-                        " - publish skipped")
+                              " - publish skipped")
                 continue
 
             # Compare our unsigned one with the downloaded one...
             compare_result = common.compare_apks(srcapk, apkfile, tmp_dir)
             if compare_result:
                 logging.error("...verfication failed - publish skipped : "
-                        + compare_result)
+                              + compare_result)
                 continue
 
             # Success! So move the downloaded file to the repo...
diff --git a/fdroidserver/verify.py b/fdroidserver/verify.py
index 041b5cc7..9139d37a 100644
--- a/fdroidserver/verify.py
+++ b/fdroidserver/verify.py
@@ -83,9 +83,9 @@ def main():
                 raise FDroidException("Failed to get " + apkfilename)
 
             compare_result = common.compare_apks(
-                    os.path.join(unsigned_dir, apkfilename),
-                    remoteapk,
-                    tmp_dir)
+                os.path.join(unsigned_dir, apkfilename),
+                remoteapk,
+                tmp_dir)
             if compare_result:
                 raise FDroidException(compare_result)