Refactor gradle ensure tasks

Author: albin-mullvad

.github/workflows/android-app.yml

@@ -114,7 +114,7 @@ jobs:
         uses: actions/cache@v4
         id: cache-relay-list
         with:
-          path: build/relays.json
+          path: android/app/build/extraAssets/relays.json
           key: relay-list-${{ steps.get-date.outputs.date }}
 
       - name: Checkout repository
@@ -128,14 +128,14 @@ jobs:
         env:
           RUSTFLAGS: --deny warnings
         run: |
-          mkdir -p build
-          cargo run --bin relay_list > build/relays.json
+          mkdir -p android/app/build/extraAssets
+          cargo run --bin relay_list > android/app/build/extraAssets/relays.json
 
       - name: Upload
         uses: actions/upload-artifact@v4
         with:
           name: relay-list
-          path: build/relays.json
+          path: android/app/build/extraAssets/relays.json
           if-no-files-found: error
           retention-days: 7
 
@@ -313,7 +313,6 @@ jobs:
       - uses: actions/download-artifact@v4
         with:
           name: relay-list
-          path: build
 
       - name: Build app
         uses: burrunan/gradle-cache-action@v1

android/app/build.gradle.kts

@@ -19,6 +19,7 @@ plugins {
 
 val repoRootPath = rootProject.projectDir.absoluteFile.parentFile.absolutePath
 val extraAssetsDirectory = "${project.buildDir}/extraAssets"
+val relayListPath = "$extraAssetsDirectory/relays.json"
 val defaultChangelogAssetsDirectory = "$repoRootPath/android/src/main/play/release-notes/"
 val extraJniDirectory = "${project.buildDir}/extraJni"
 
@@ -146,10 +147,6 @@ android {
             )
     }
 
-    tasks.withType<MergeSourceSetFolders> { dependsOn(getTasksByName("copyExtraAssets", true)) }
-
-    tasks.withType<LintModelWriterTask> { dependsOn(getTasksByName("copyExtraAssets", true)) }
-
     // Suppressing since we don't seem have much of an option than using this api. The impact should
     // also be limited to tests.
     @Suppress("UnstableApiUsage")
@@ -238,10 +235,14 @@ android {
             }
 
         createDistBundle.dependsOn("bundle$capitalizedVariantName")
-    }
 
-    project.tasks.assemble.dependsOn("ensureJniDirectoryExist")
-    project.tasks.assemble.dependsOn("ensureValidVersionCode")
+        // Ensure all relevant assemble tasks depend on our ensure tasks.
+        tasks.get("assemble$capitalizedVariantName").apply {
+            dependsOn(tasks.get("ensureRelayListExist"))
+            dependsOn(tasks.get("ensureJniDirectoryExist"))
+            dependsOn(tasks.get("ensureValidVersionCode"))
+        }
+    }
 }
 
 junitPlatform {
@@ -274,14 +275,16 @@ configure<org.owasp.dependencycheck.gradle.extension.DependencyCheckExtension> {
     skipConfigurations = listOf("lintClassPath")
 }
 
-tasks.register("copyExtraAssets", Copy::class) {
-    from("$repoRootPath/build")
-    include("relays.json")
-    into(extraAssetsDirectory)
+tasks.register("ensureRelayListExist") {
+    doLast {
+        if (!file(relayListPath).exists()) {
+            throw GradleException("Missing relay list: $relayListPath")
+        }
+    }
 }
 
 tasks.register("ensureJniDirectoryExist") {
-    doFirst {
+    doLast {
         if (!file(extraJniDirectory).exists()) {
             throw GradleException("Missing JNI directory: $extraJniDirectory")
         }
@@ -306,12 +309,6 @@ tasks.create("printVersion") {
     }
 }
 
-afterEvaluate {
-    tasks.withType(com.android.build.gradle.internal.lint.AndroidLintAnalysisTask::class.java) {
-        mustRunAfter(tasks.getByName("copyExtraAssets"))
-    }
-}
-
 play { serviceAccountCredentials.set(file("play-api-key.json")) }
 
 dependencies {

build-apk.sh

@@ -73,6 +73,7 @@ else
 fi
 
 $GRADLE_CMD --console plain clean
+mkdir -p "app/build/extraAssets"
 mkdir -p "app/build/extraJni"
 popd
 
@@ -111,7 +112,7 @@ for ARCHITECTURE in ${ARCHITECTURES:-aarch64 armv7 x86_64 i686}; do
 done
 
 echo "Updating relays.json..."
-cargo run --bin relay_list "${CARGO_ARGS[@]}" > build/relays.json
+cargo run --bin relay_list "${CARGO_ARGS[@]}" > android/build/extraAssets/relays.json
 
 cd "$SCRIPT_DIR/android"
 $GRADLE_CMD --console plain "${GRADLE_TASKS[@]}"