From: Ciaran Gultnieks <ciaran@ciarang.com>
Date: Thu, 28 Jun 2012 11:52:35 +0000 (+0100)
Subject: More maven build output recognition
X-Git-Tag: 0.1~802
X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=commitdiff_plain;h=712531ba37cadfe01a8f6d23b6fe2ecf62c3aa11;p=fdroidserver.git

More maven build output recognition
---

diff --git a/fdroidserver/build.py b/fdroidserver/build.py
index d6d7e398..d7a7144e 100644
--- a/fdroidserver/build.py
+++ b/fdroidserver/build.py
@@ -201,12 +201,15 @@ def build_local(app, thisbuild, vcs, build_dir, output_dir, extlib_dir, tmp_dir,
                 thisbuild['version'] + "-unsigned.apk")
         src = os.path.join(bindir, src)
     elif thisbuild.has_key('maven'):
-        try:
-            src = re.match(r".*^\[INFO\] .*apkbuilder.*/([^/]*)\.apk",
-                output, re.S|re.M).group(1)
-        except:
+        m = re.match(r".*^\[INFO\] .*apkbuilder.*/([^/]*)\.apk",
+                output, re.S|re.M)
+        if not m:
+            m = re.match(r".*^\[INFO\] Creating additional unsigned apk file .*/([^/]+)\.apk",
+                    output, re.S|re.M)
+        if not m:
             print output
             raise BuildException('Failed to find output')
+        src = m.group(1)
         src = os.path.join(bindir, src) + '.apk'
 #[INFO] Installing /home/ciaran/fdroidserver/tmp/mainline/application/target/callerid-1.0-SNAPSHOT.apk
     else:
@@ -217,6 +220,8 @@ def build_local(app, thisbuild, vcs, build_dir, output_dir, extlib_dir, tmp_dir,
     # By way of a sanity check, make sure the version and version
     # code in our new apk match what we expect...
     print "Checking " + src
+    if not os.path.exists(src):
+        raise BuildException("Unsigned apk is not at expected location of " + src)
     p = subprocess.Popen([os.path.join(sdk_path, 'platform-tools',
                                     'aapt'),
                         'dump', 'badging', src],