android envsetup: Stop exporting (and honoring) ANDROID_SDK_VERSION.

If someone wants to use a custom android sdk version, they now need to set
the android_sdk_version gyp define instead of this env var.

(Depends on one internal change to an internal bot that currently sets the
env var: https://chrome-internal-review.googlesource.com/154255)

BUG=330631

Review URL: https://codereview.chromium.org/159463002

git-svn-id: http://src.chromium.org/svn/trunk/src/build@251354 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
This commit is contained in:
thakis@chromium.org 2014-02-14 16:59:48 +00:00
Родитель 43f7a49bbc
Коммит 3e3ae4cdf8
2 изменённых файлов: 4 добавлений и 9 удалений

Просмотреть файл

@ -108,11 +108,6 @@ sdk_build_init() {
if [[ -z "${ANDROID_NDK_ROOT}" || ! -d "${ANDROID_NDK_ROOT}" ]]; then if [[ -z "${ANDROID_NDK_ROOT}" || ! -d "${ANDROID_NDK_ROOT}" ]]; then
export ANDROID_NDK_ROOT="${CHROME_SRC}/third_party/android_tools/ndk/" export ANDROID_NDK_ROOT="${CHROME_SRC}/third_party/android_tools/ndk/"
fi fi
if [[ -z "${ANDROID_SDK_VERSION}" ]]; then
export ANDROID_SDK_VERSION=19
else
sdk_defines+=" android_sdk_version=${ANDROID_SDK_VERSION}"
fi
if [[ -z "${ANDROID_SDK_ROOT}" || ! -d "${ANDROID_SDK_ROOT}" ]]; then if [[ -z "${ANDROID_SDK_ROOT}" || ! -d "${ANDROID_SDK_ROOT}" ]]; then
export ANDROID_SDK_ROOT="${CHROME_SRC}/third_party/android_tools/sdk/" export ANDROID_SDK_ROOT="${CHROME_SRC}/third_party/android_tools/sdk/"
else else
@ -148,12 +143,12 @@ ${ANDROID_SDK_BUILD_TOOLS_VERSION}"
############################################################################# #############################################################################
webview_build_init() { webview_build_init() {
# Use the latest API in the AOSP prebuilts directory (change with AOSP roll). # Use the latest API in the AOSP prebuilts directory (change with AOSP roll).
export ANDROID_SDK_VERSION=18 android_sdk_version=18
# For the WebView build we always use the NDK and SDK in the Android tree. # For the WebView build we always use the NDK and SDK in the Android tree.
export ANDROID_NDK_ROOT=${ANDROID_BUILD_TOP}/prebuilts/ndk/8 export ANDROID_NDK_ROOT=${ANDROID_BUILD_TOP}/prebuilts/ndk/8
export ANDROID_SDK_ROOT=${ANDROID_BUILD_TOP}/prebuilts/sdk/\ export ANDROID_SDK_ROOT=${ANDROID_BUILD_TOP}/prebuilts/sdk/\
${ANDROID_SDK_VERSION} ${android_sdk_version}
common_vars_defines common_vars_defines
@ -173,7 +168,7 @@ ${ANDROID_SDK_VERSION}
DEFINES+=" android_sdk=\$(PWD)/${ANDROID_SDK}" DEFINES+=" android_sdk=\$(PWD)/${ANDROID_SDK}"
DEFINES+=" android_sdk_root=\$(PWD)/${ANDROID_SDK}" DEFINES+=" android_sdk_root=\$(PWD)/${ANDROID_SDK}"
DEFINES+=" android_sdk_tools=\$(PWD)/${ANDROID_SDK_TOOLS}" DEFINES+=" android_sdk_tools=\$(PWD)/${ANDROID_SDK_TOOLS}"
DEFINES+=" android_sdk_version=${ANDROID_SDK_VERSION}" DEFINES+=" android_sdk_version=${android_sdk_version}"
DEFINES+=" android_toolchain=${ANDROID_TOOLCHAIN}" DEFINES+=" android_toolchain=${ANDROID_TOOLCHAIN}"
if [[ -n "$CHROME_ANDROID_WEBVIEW_OFFICIAL_BUILD" ]]; then if [[ -n "$CHROME_ANDROID_WEBVIEW_OFFICIAL_BUILD" ]]; then
DEFINES+=" logging_like_official_build=1" DEFINES+=" logging_like_official_build=1"

Просмотреть файл

@ -1389,7 +1389,7 @@
'android_sdk_root%': '<!(cd <(DEPTH) && pwd -P)/third_party/android_tools/sdk/', 'android_sdk_root%': '<!(cd <(DEPTH) && pwd -P)/third_party/android_tools/sdk/',
'android_host_arch%': '<!(uname -m)', 'android_host_arch%': '<!(uname -m)',
# Android API-level of the SDK used for compilation. # Android API-level of the SDK used for compilation.
'android_sdk_version%': '<!(/bin/echo -n ${ANDROID_SDK_VERSION})', 'android_sdk_version%': '19',
# Android SDK build tools (e.g. dx, aapt, aidl) # Android SDK build tools (e.g. dx, aapt, aidl)
'android_sdk_tools%': '<!(/bin/echo -n ${ANDROID_SDK_TOOLS})', 'android_sdk_tools%': '<!(/bin/echo -n ${ANDROID_SDK_TOOLS})',
'host_os%': "<!(uname -s | sed -e 's/Linux/linux/;s/Darwin/mac/')", 'host_os%': "<!(uname -s | sed -e 's/Linux/linux/;s/Darwin/mac/')",