From 2b0badd4c43c04eebe3bf2a7057fac6fe9b1b90d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Daniel=20Mart=C3=AD?= <mvdan@mvdan.cc>
Date: Tue, 5 Nov 2013 09:26:26 +0100
Subject: [PATCH] Fix init with common, don't require cwd in FDroidPopen

---
 fdroidserver/common.py |  5 +++--
 fdroidserver/init.py   | 28 +++++++++++++---------------
 2 files changed, 16 insertions(+), 17 deletions(-)

diff --git a/fdroidserver/common.py b/fdroidserver/common.py
index fd84d7c5..a92ef6b9 100644
--- a/fdroidserver/common.py
+++ b/fdroidserver/common.py
@@ -1910,7 +1910,7 @@ class PopenResult:
     stderr = ''
     stdout_apk = ''
 
-def FDroidPopen(commands, cwd):
+def FDroidPopen(commands, cwd=None):
     """
     Runs a command the FDroid way and returns return code and output
 
@@ -1918,7 +1918,8 @@ def FDroidPopen(commands, cwd):
     """
 
     if options.verbose:
-        print "Directory: %s" % cwd
+        if cwd is not None:
+            print "Directory: %s" % cwd
         print " > %s" % ' '.join(commands)
 
     result = PopenResult()
diff --git a/fdroidserver/init.py b/fdroidserver/init.py
index 37ece55c..82e533ed 100644
--- a/fdroidserver/init.py
+++ b/fdroidserver/init.py
@@ -29,6 +29,7 @@ import sys
 from optparse import OptionParser
 
 import common
+from common import FDroidPopen, BuildException
 
 
 config = {}
@@ -56,24 +57,21 @@ def genpassword():
 def genkey(keystore, repo_keyalias, password, keydname):
     '''generate a new keystore with a new key in it for signing repos'''
     print('Generating a new key in "' + keystore + '"...')
-    p = subprocess.Popen(['keytool', '-genkey',
-                          '-keystore', keystore, '-alias', repo_keyalias,
-                          '-keyalg', 'RSA', '-keysize', '4096',
-                          '-sigalg', 'SHA256withRSA',
-                          '-validity', '10000',
-                          '-storepass', password, '-keypass', password,
-                          '-dname', keydname],
-                         stdout=subprocess.PIPE)
-    output = p.communicate()[0]
-    print(output)
+    p = FDroidPopen(['keytool', '-genkey',
+                '-keystore', keystore, '-alias', repo_keyalias,
+                '-keyalg', 'RSA', '-keysize', '4096',
+                '-sigalg', 'SHA256withRSA',
+                '-validity', '10000',
+                '-storepass', password, '-keypass', password,
+                '-dname', keydname])
     if p.returncode != 0:
-        raise BuildException("Failed to generate key")
+        raise BuildException("Failed to generate key", p.stdout, p.stderr)
     # now show the lovely key that was just generated
     p = subprocess.Popen(['keytool', '-list', '-v',
-                          '-keystore', keystore, '-alias', repo_keyalias],
-                         stdin=subprocess.PIPE,
-                         stdout=subprocess.PIPE,
-                         stderr=subprocess.PIPE)
+                '-keystore', keystore, '-alias', repo_keyalias],
+                stdin=subprocess.PIPE,
+                stdout=subprocess.PIPE,
+                stderr=subprocess.PIPE)
     output = p.communicate(password)[0]
     print(output.lstrip().strip() + '\n\n')