Move APK building into entrypoint

The APK is no longer built by building the docker image but by running
the docker container. The docker image only provides a build environment
for fairemail and is re-usable across multiple builds.

Dockerfile

@@ -15,22 +15,6 @@ RUN mkdir "$ANDROID_HOME" .android \
     && $ANDROID_HOME/tools/bin/sdkmanager --update \
     && yes | $ANDROID_HOME/tools/bin/sdkmanager --licenses
 
-ADD patches /patches
+ADD entrypoint.sh /entrypoint.sh
 
-RUN mkdir /apk
+ENTRYPOINT ["/bin/sh", "/entrypoint.sh"]
-
-WORKDIR /app
-
-ARG VERSION
-ARG KEYSTORE_FILE
-ARG KEYSTORE_PROPERTIES
-
-RUN curl -OL https://github.com/M66B/FairEmail/archive/$VERSION.tar.gz \
-    && tar xvzf $VERSION.tar.gz \
-    && cd FairEmail-$VERSION \
-    && echo $KEYSTORE_FILE | base64 -d > keystore.jks \
-    && echo $KEYSTORE_PROPERTIES | base64 -d > keystore.properties
-RUN cd FairEmail-$VERSION \
-    && (for patch in /patches/*; do patch -p0 < $patch; done) \
-    && gradle --no-daemon assemble
-RUN cp FairEmail-$VERSION/app/build/outputs/apk/github/release/FairEmail-v$VERSION-github-release.apk /apk

build.sh

@@ -19,23 +19,21 @@ if rsync sillywalk.de:/srv/www/sillywalk.de/root/releases/$APK_FILENAME; then
     exit 1
 fi
 
-docker build \
+KEYSTORE_FILE=$(cat $ANDROID_KEYSTORE_FILE | base64 -w0) \
-    --build-arg VERSION=$VERSION \
+KEYSTORE_PROPERTIES=$(envsubst < templates/keystore.properties | base64 -w0) \
-    --build-arg KEYSTORE_FILE=$(cat $ANDROID_KEYSTORE_FILE | base64 -w0) \
-    --build-arg KEYSTORE_PROPERTIES=$(envsubst < templates/keystore.properties | base64 -w0) \
-    -t fairemail-apk \
-    .
-
-DOCKER_CONTAINER_ID=$(docker create fairemail-apk)
 
+mkdir -p apk
 trap "{
-    rm -f $APK_FILENAME;
+    rm -rf apk;
-    docker rm $DOCKER_CONTAINER_ID;
 }" EXIT
 
-docker cp $DOCKER_CONTAINER_ID:/apk/$APK_FILENAME .
+docker build -t fairemail-apk-builder .
+docker run --rm -it \
+    -v $(pwd)/patches:/patches:ro \
+    -v $(pwd)/apk:/apk:rw \
+    fairemail-apk-builder $VERSION $KEYSTORE_FILE $KEYSTORE_PROPERTIES
 
-UPLOADED=$(rsync -i --ignore-existing $APK_FILENAME sillywalk.de:/srv/www/sillywalk.de/root/releases)
+UPLOADED=$(rsync -i --ignore-existing apk/$APK_FILENAME sillywalk.de:/srv/www/sillywalk.de/root/releases)
 
 if [ -z "$UPLOADED" ]; then
     echo "$VERSION already uploaded. Aborting."

entrypoint.sh

@@ -0,0 +1,23 @@
+#!/bin/sh
+
+set -eu
+
+VERSION=$1
+KEYSTORE_FILE=$2
+KEYSTORE_PROPERTIES=$3
+
+curl -OL https://github.com/M66B/FairEmail/archive/$VERSION.tar.gz
+tar xvzf $VERSION.tar.gz
+
+cd FairEmail-$VERSION
+
+echo $KEYSTORE_FILE | base64 -d > keystore.jks
+echo $KEYSTORE_PROPERTIES | base64 -d > keystore.properties
+
+for patch in /patches/*; do
+    patch -p0 < $patch
+done
+
+gradle --no-daemon assemble
+
+cp app/build/outputs/apk/github/release/FairEmail-v$VERSION-github-release.apk /apk

patches/10-pro.diff

@@ -1,12 +1,13 @@
---- app/src/main/java/eu/faircode/email/ActivityBilling.java	2020-04-11 09:27:44.000000000 +0000
+--- app/src/play/java/eu/faircode/email/ActivityBilling.java	2020-08-15 10:33:56.000000000 +0000
-+++ app/src/main/java/eu/faircode/email/ActivityBilling.java	2020-04-11 09:27:44.000000000 +0000
++++ app/src/play/java/eu/faircode/email/ActivityBilling.java	2020-08-15 10:33:56.000000000 +0000
-@@ -192,9 +192,7 @@
+@@ -192,10 +192,7 @@
      }
  
      static boolean isPro(Context context) {
 -        if (BuildConfig.DEBUG && false)
 -            return true;
--        return PreferenceManager.getDefaultSharedPreferences(context).getBoolean("pro", false);
+-        return PreferenceManager.getDefaultSharedPreferences(context)
+-                .getBoolean("pro", false);
 +        return true;
      }