fix: ANDROID_SDK_ROOT variable (#951)
authorNorman Breau <norman@normanbreau.com>
Thu, 16 Apr 2020 02:43:17 +0000 (23:43 -0300)
committerGitHub <noreply@github.com>
Thu, 16 Apr 2020 02:43:17 +0000 (23:43 -0300)
commit16a88ec631f97dd73a4f37a0dacc35bffed13547
tree4c2b2cc25da0170b2cbb84bb48d7ae87b56a022d
parent1ce2b6b31500e38a195d9a6e18cfe1d48f27df81
fix: ANDROID_SDK_ROOT variable (#951)

This commit does the following:
- Makes ANDROID_SDK_ROOT the primary variable to look for the Android SDK location.
- Makes ANDROID_HOME the fallback variable, if ANDROID_SDK_ROOT is not present/valid.

Gradle updates:
Note that the following gradle updates were required, otherwise the android gradle plugin did not honour the ANDROID_SDK_ROOT variable.

- Updates the framework's android studio's gradle plugin from version 3.3.0 to 3.5.3.
Not only this is required for android's gradle to obey ANDROID_SDK_ROOT, it is now in sync with the Android test project/
- Updates the Androidx test project to use gralde plugin from version 3.3.0 to 3.5.3, to match Android Test & framework.
- Consequentially, this required to also upgrade AndroidX test project to use Gradle 6.1, which also matches both the Android test project & framework

These changes above fixes #949

Additionally, since we update the environment variables dynamically, the environment variable printout produced misleading information.
The environment variable printout will now print out the variable as defined by the user (before the tooling messes with them). An additional log
is printed that tells the user exactly what Cordova is going to use for the Android SDK path. This should fix #670
bin/templates/cordova/lib/check_reqs.js
framework/cordova.gradle
spec/unit/check_reqs.spec.js