Commit 5919b7e1 authored by thakis@chromium.org's avatar thakis@chromium.org

android envsetup: Make passing --target-arch an error.

This is to make sure I found all the bots that used to pass this. I'll
completely remove the flag in a day or two.

Also remove the function for printing usage information. I accidentally removed
processing for --help in r252034, but having a --help flag for printing --help
as only supported flag is a bit silly anyways, so get rid of this.

(Depends on https://codereview.chromium.org/170783015/
and https://chrome-internal-review.googlesource.com/#/c/155292/)

BUG=330631
R=yfriedman@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@252166 0039d316-1c4b-4281-b951-d872f2087c98
parent 17d0de9a
...@@ -22,7 +22,9 @@ SCRIPT_DIR="$(dirname "${BASH_SOURCE:-$0}")" ...@@ -22,7 +22,9 @@ SCRIPT_DIR="$(dirname "${BASH_SOURCE:-$0}")"
export ANDROID_SDK_BUILD=1 # Default to SDK build. export ANDROID_SDK_BUILD=1 # Default to SDK build.
process_options "$@" if ! process_options "$@" ; then
return 1
fi
# When building WebView as part of Android we can't use the SDK. Other builds # When building WebView as part of Android we can't use the SDK. Other builds
# default to using the SDK. # default to using the SDK.
......
...@@ -42,14 +42,6 @@ common_vars_defines() { ...@@ -42,14 +42,6 @@ common_vars_defines() {
} }
################################################################################
# Prints out help message on usage.
################################################################################
print_usage() {
echo "usage: ${0##*/} [--help]" >& 2
echo "--help this help" >& 2
}
################################################################################ ################################################################################
# Process command line options. # Process command line options.
################################################################################ ################################################################################
...@@ -57,9 +49,10 @@ process_options() { ...@@ -57,9 +49,10 @@ process_options() {
while [[ -n $1 ]]; do while [[ -n $1 ]]; do
case "$1" in case "$1" in
--target-arch=*) --target-arch=*)
echo "WARNING: --target-arch is ignored and will be an error soon." echo "ERROR: --target-arch is ignored."
echo "Pass -Dtarget_arch=foo to gyp instead." echo "Pass -Dtarget_arch=foo to gyp instead."
echo "(x86 is spelled ia32 in gyp, mips becomes mipsel, arm stays arm)" echo "(x86 is spelled ia32 in gyp, mips becomes mipsel, arm stays arm)"
return 1
;; ;;
*) *)
# Ignore other command line options # Ignore other command line options
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment