From 15e49571bf9684f84609d4c31e3045c45fbb868c Mon Sep 17 00:00:00 2001 From: Dale Curtis Date: Wed, 13 Oct 2010 12:38:04 -0700 Subject: [PATCH] Fixed incorrect option comment in run_remote_tests. Autotest will not split comma separated arguments. The correct calling method for multiple arguments is to separate them by spaces and quote the whole thing. E.g., instead of: ./run_remote_tests --args arg1,arg2 It should be: ./run_remote_tests --args "arg1 arg2" See line 140 in src/third_party/autotest/files/server/autoserv_parser.py for confirmation. Change-Id: I88d65c0bd144942a856c2f95371d5a55b0a3a172 BUG= TEST=No code changes. Ran --help and --args to verify nothing dumb happened and the correctness of comment. Review URL: http://codereview.chromium.org/3772002 --- run_remote_tests.sh | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/run_remote_tests.sh b/run_remote_tests.sh index 36e028d2af..64ddaf295a 100755 --- a/run_remote_tests.sh +++ b/run_remote_tests.sh @@ -14,7 +14,8 @@ get_default_board -DEFINE_string args "" "Command line arguments for test, separated with comma" a +DEFINE_string args "" \ + "Command line arguments for test. Quoted and space separated if multiple." a DEFINE_string board "$DEFAULT_BOARD" \ "The board for which you are building autotest" DEFINE_string chroot "${DEFAULT_CHROOT_DIR}" "alternate chroot location" c