drop x86 and x86_64

Signed-off-by: Jakub Sokołowski <jakub@status.im>
status-im · Jun 9, 2019 · 71a6d90 · 71a6d90
1 parent d75a412
commit 71a6d90
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 11 deletions.
diff --git a/android/app/build.gradle b/android/app/build.gradle
@@ -149,8 +149,6 @@ android {
         targetCompatibility JavaVersion.VERSION_1_8
     }
 
-
-
     defaultConfig {
         applicationId "im.status.ethereum"
         minSdkVersion 23
@@ -159,7 +157,7 @@ android {
         versionCode getVersionCode()
         versionName getVersionName()
         ndk {
-            abiFilters "armeabi-v7a", "arm64-v8a", "x86", "x86-64"
+            abiFilters getEnvOrConfig('NDK_ABI_FILTERS').split(';')
         }
     }
     /**
@@ -179,7 +177,6 @@ android {
         exclude 'META-INF/rxjava.properties'
         exclude '/lib/mips64/**'
         exclude '/lib/arm64-v8a/**'
-        exclude '/lib/x86_64/**'
         /** Fix for: Execution failed for task ':app:transformNativeLibsWithStripDebugSymbolForDebug'.
         *   with recent version of ndk (17.0.4754217)
         */
@@ -204,7 +201,7 @@ android {
             reset()
             enable enableSeparateBuildPerCPUArchitecture
             universalApk false  // If true, also generate a universal APK
-            include "armeabi-v7a", "x86"
+            include "armeabi-v7a", "arm64-v8a", "x86"
         }
     }
     buildTypes {

diff --git a/android/gradle.properties b/android/gradle.properties
@@ -24,4 +24,7 @@ STATUS_RELEASE_STORE_PASSWORD=password
 STATUS_RELEASE_KEY_ALIAS=status
 STATUS_RELEASE_KEY_PASSWORD=password
 
+# platforms for which to build the Android bundle
+NDK_ABI_FILTERS=armeabi-v7a;arm64-v8a;x86
+
 org.gradle.jvmargs=-Xmx8704M
diff --git a/ci/android.groovy b/ci/android.groovy
@@ -6,13 +6,19 @@ def bundle() {
   /* Disable Gradle Daemon https://stackoverflow.com/questions/38710327/jenkins-builds-fail-using-the-gradle-daemon */
   def gradleOpt = "-PbuildUrl='${currentBuild.absoluteUrl}' -Dorg.gradle.daemon=false "
   def target = "release"
+  /* we don't need x86 for any builds except e2e */
+  env.NDK_ABI_FILTERS="armeabi-v7a;arm64-v8a"
 
-  if (params.BUILD_TYPE == 'pr') {
-    /* PR builds shouldn't replace normal releases */
-    target = 'pr'
-  } else if (btype == 'release') {
-    gradleOpt += "-PreleaseVersion='${utils.getVersion()}'"
+  switch (btype) {
+    case 'pr': /* PR builds shouldn't replace normal releases */
+      target = 'pr'; break;
+    case 'e2e':
+      target = 'pr';
+      env.NDK_ABI_FILTERS="x86"; break
+    case 'release':
+      gradleOpt += "-PreleaseVersion='${utils.getVersion('mobile_files')}'"
   }
+
   dir('android') {
     withCredentials([
       string(
@@ -28,7 +34,7 @@ def bundle() {
       nix.shell(
         "./gradlew assemble${target.capitalize()} ${gradleOpt}",
         keep: [
-          'REALM_DISABLE_ANALYTICS',
+          'REALM_DISABLE_ANALYTICS', 'NDK_ABI_FILTERS',
           'STATUS_RELEASE_STORE_FILE', 'STATUS_RELEASE_STORE_PASSWORD',
           'STATUS_RELEASE_KEY_ALIAS', 'STATUS_RELEASE_KEY_PASSWORD'
         ]