Fix Android build sequencing

Andrei Shikov · kelset · commit 121a6a49c66e · 2021-06-18T09:22:08.000+01:00
Summary:
The native libraries are compiled outside of the usual Android build flow using separate CLI task. Because of that, shared native libraries may not exist when AAR is bundled, resulting in weird sequencing issues.

This change updates gradle dependency graph, executing RN native build before Android part (as it is done in RNTester already).

Changelog: [Internal]

Reviewed By: mdvacca

Differential Revision: D29209249

fbshipit-source-id: 36386c78996b1cd9b1731735e36e571199e9e81b
diff --git a/ReactAndroid/build.gradle b/ReactAndroid/build.gradle
@@ -416,11 +416,7 @@ android {
         }
     }
 
-    tasks.withType(JavaCompile) {
-        compileTask ->
-            compileTask.dependsOn(packageReactNdkLibs)
-    }
-
+    preBuild.dependsOn(packageReactNdkLibs)
     clean.dependsOn(cleanReactNdkLib)
 
     lintOptions {

Original file line number	Diff line number	Diff line change
`@@ -416,11 +416,7 @@ android {`
`416`	`416`	`}`
`417`	`417`	`}`
`418`	`418`
`419`		`- tasks.withType(JavaCompile) {`
`420`		`- compileTask ->`
`421`		`- compileTask.dependsOn(packageReactNdkLibs)`
`422`		`- }`
`423`		`-`
	`419`	`+ preBuild.dependsOn(packageReactNdkLibs)`
`424`	`420`	`clean.dependsOn(cleanReactNdkLib)`
`425`	`421`
`426`	`422`	`lintOptions {`