Skip to content

Commit aed9c1d

Browse files
committed
Remove logic to set ANDROID_SDK_HOME
1 parent bde361c commit aed9c1d

File tree

1 file changed

+2
-9
lines changed

1 file changed

+2
-9
lines changed

scripts/gha/build_testapps.py

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -59,13 +59,11 @@
5959
6060
JAVA_HOME=/Library/Java/JavaVirtualMachines/jdk-8-latest/Contents/Home
6161
ANDROID_HOME=/Users/user_name/Library/Android/sdk
62-
ANDROID_SDK_HOME=/Users/user_name/Library/Android/sdk
6362
ANDROID_NDK_HOME=/Users/user_name/Library/Android/sdk/ndk-bundle
6463
6564
Or on Linux:
6665
JAVA_HOME=/usr/local/buildtools/java/jdk/
6766
ANDROID_HOME=~/Android/Sdk
68-
ANDROID_SDK_HOME=~/Android/Sdk
6967
ANDROID_NDK_HOME=~/Android/Sdk/ndk
7068
7169
If using this tool frequently, you will likely find it convenient to
@@ -97,7 +95,6 @@
9795
# Environment variables
9896
_JAVA_HOME = "JAVA_HOME"
9997
_ANDROID_HOME = "ANDROID_HOME"
100-
_ANDROID_SDK_HOME = "ANDROID_SDK_HOME"
10198
_NDK_ROOT = "NDK_ROOT"
10299
_ANDROID_NDK_HOME = "ANDROID_NDK_HOME"
103100

@@ -538,12 +535,8 @@ def _validate_android_environment_variables():
538535
android_home = os.environ.get(_ANDROID_HOME)
539536
if not os.environ.get(_JAVA_HOME):
540537
logging.warning("%s not set", _JAVA_HOME)
541-
if not os.environ.get(_ANDROID_SDK_HOME):
542-
if android_home: # Use ANDROID_HOME as backup for ANDROID_SDK_HOME
543-
os.environ[_ANDROID_SDK_HOME] = android_home
544-
logging.info("%s not found, using %s", _ANDROID_SDK_HOME, _ANDROID_HOME)
545-
else:
546-
logging.warning("Missing: %s and %s", _ANDROID_SDK_HOME, _ANDROID_HOME)
538+
if not android_home:
539+
logging.warning("Missing ANDROID_HOME: %s", _ANDROID_HOME)
547540
# Different environments may have different NDK env vars specified. We look
548541
# for these, in this order, and set the others to the first found.
549542
# If none are set, we check the default location for the ndk.

0 commit comments

Comments
 (0)