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

Unified Diff: run_remote_tests.sh

Issue 3790005: crosutil: No longer use autotest-0.0.1 in run_remote_tests even outside chroot (Closed) Base URL: http://git.chromium.org/git/crosutils.git
Patch Set: Created 10 years, 2 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 | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: run_remote_tests.sh
diff --git a/run_remote_tests.sh b/run_remote_tests.sh
index 36e028d2afddb250a610f7723354ec55fcf1ed1c..dd62f51b7d15c400d061aee749092146285ef6d8 100755
--- a/run_remote_tests.sh
+++ b/run_remote_tests.sh
@@ -20,7 +20,6 @@ DEFINE_string board "$DEFAULT_BOARD" \
DEFINE_string chroot "${DEFAULT_CHROOT_DIR}" "alternate chroot location" c
DEFINE_boolean cleanup ${FLAGS_FALSE} "Clean up temp directory"
DEFINE_integer iterations 1 "Iterations to run every top level test" i
-DEFINE_string prepackaged_autotest "" "Use this prepackaged autotest dir"
DEFINE_string results_dir_root "" "alternate root results directory"
DEFINE_boolean verbose ${FLAGS_FALSE} "Show verbose autoserv output" v
@@ -155,17 +154,8 @@ function main() {
remote_access_init
- local autotest_dir=""
- if [[ -z "${FLAGS_prepackaged_autotest}" ]]; then
- learn_board
- if [[ -n "${CROS_WORKON_SRCROOT}" ]]; then
- autotest_dir="/build/${FLAGS_board}/usr/local/autotest"
- else
- autotest_dir="${GCLIENT_ROOT}/src/third_party/autotest/files"
- fi
- else
- autotest_dir="${FLAGS_prepackaged_autotest}"
- fi
+ learn_board
+ autotest_dir="${FLAGS_chroot}/build/${FLAGS_board}/usr/local/autotest"
local control_files_to_run=""
local chrome_autotests="${CHROME_ROOT}/src/chrome/test/chromeos/autotest/files"
@@ -252,14 +242,11 @@ function main() {
RAN_ANY_TESTS=${FLAGS_TRUE}
- # HACK: Temporary hack for cros-workon conversion
- [[ -n "${CROS_WORKON_SRCROOT}" ]] && WORKON_SUFFIX=_workon
-
local enter_chroot=""
- local autotest="${GCLIENT_ROOT}/src/scripts/autotest${WORKON_SUFFIX}"
+ local autotest="${GCLIENT_ROOT}/src/scripts/autotest_workon"
if [[ ${INSIDE_CHROOT} -eq 0 ]]; then
enter_chroot="./enter_chroot.sh --chroot ${FLAGS_chroot} --"
- autotest="./autotest${WORKON_SUFFIX}"
+ autotest="./autotest_workon"
fi
# Remove chrome autotest location prefix from control_file if needed
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698