Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(397)

Unified Diff: src/scripts/build_autotest.sh

Issue 661203: Modify scripts to be more informative and add autox/autologin for build script (Closed)
Patch Set: Created 10 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | src/scripts/mod_for_test_scripts/350addTestAuthKeys » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/scripts/build_autotest.sh
diff --git a/src/scripts/build_autotest.sh b/src/scripts/build_autotest.sh
index 2c0472b9b4240d20fa1f4f0e888abbd898439a01..f9bac5e13e2c9d562911b02812f85d03b36ba768 100755
--- a/src/scripts/build_autotest.sh
+++ b/src/scripts/build_autotest.sh
@@ -21,7 +21,8 @@ DEFAULT_TESTS_LIST="all"
DEFINE_string build "${DEFAULT_TESTS_LIST}" \
"a comma seperated list of autotest client tests to be prebuilt." b
-DEFINE_boolean prompt $FLAGS_TRUE "Prompt user when building all tests"
+DEFINE_boolean prompt $FLAGS_TRUE "Prompt user when building all tests."
+DEFINE_boolean autox $FLAGS_TRUE "Build autox along with autotest for autologin"
petkov 2010/02/26 21:50:39 maybe remove "for autologin"
# More useful help
FLAGS_HELP="usage: $0 [flags]"
@@ -64,5 +65,12 @@ else
TEST_LIST=${FLAGS_build}
fi
+# Decide whether or not to build autox and set use flag
+if [ $FLAGS_autox -eq "$FLAGS_TRUE" ] ; then
+ USE=
+else
+ USE=-autox
+fi
+
GCLIENT_ROOT="${GCLIENT_ROOT}" TEST_LIST=${TEST_LIST} \
-"emerge-${FLAGS_board}" chromeos-base/autotest
+ USE="$USE" "emerge-${FLAGS_board}" chromeos-base/autotest
« no previous file with comments | « no previous file | src/scripts/mod_for_test_scripts/350addTestAuthKeys » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698