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

Unified Diff: src/scripts/run_remote_tests.sh

Issue 556060: Enable building autotests in test loop and sbt script (Closed)
Patch Set: fix 80 chars Created 10 years, 11 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/sync_build_test.sh » ('j') | src/scripts/sync_build_test.sh » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/scripts/run_remote_tests.sh
diff --git a/src/scripts/run_remote_tests.sh b/src/scripts/run_remote_tests.sh
index b50ef068371ce57a32e66bf56c69658cc0c23093..24c8619285b3fc6f8a06435061a752c26848b1a3 100755
--- a/src/scripts/run_remote_tests.sh
+++ b/src/scripts/run_remote_tests.sh
@@ -22,6 +22,7 @@ DEFINE_boolean update_db ${FLAGS_FALSE} "Put results in autotest database" u
DEFINE_string machine_desc "" "Machine description used in database"
DEFINE_string build_desc "" "Build description used in database"
DEFINE_string chroot_dir "${DEFAULT_CHROOT_DIR}" "alternate chroot location" c
+DEFINE_string results_dir_root "" "alternate root results directory"
function cleanup() {
if [[ $FLAGS_cleanup -eq ${FLAGS_TRUE} ]]; then
@@ -154,6 +155,12 @@ function main() {
FLAGS_machine_desc="${FLAGS_remote}"
fi
+ if [[ -z "${FLAGS_results_dir_root}" ]]; then
+ FLAGS_results_dir_root="${TMP}"
+ fi
+
+ mkdir -p "${FLAGS_results_dir_root}"
+
for control_file in ${control_files_to_run}; do
# Assume a line starts with TEST_TYPE =
control_file=$(remove_quotes "${control_file}")
@@ -171,7 +178,8 @@ function main() {
echo "Running ${type} test ${control_file}"
local short_name=$(basename $(dirname "${control_file}"))
local start_time=$(date '+%s')
- local results_dir="${TMP}/${short_name},${FLAGS_machine_desc},${start_time}"
+ local results_dir_name="${short_name},${FLAGS_machine_desc},${start_time}"
sosa 2010/01/29 02:43:12 Unless you're planning to reuse this, I think this
+ local results_dir="${FLAGS_results_dir_root}/${results_dir_name}"
rm -rf "${results_dir}"
local verbose=""
if [[ ${FLAGS_verbose} -eq $FLAGS_TRUE ]]; then
« no previous file with comments | « no previous file | src/scripts/sync_build_test.sh » ('j') | src/scripts/sync_build_test.sh » ('J')

Powered by Google App Engine
This is Rietveld 408576698