From 30d46e3c0db3445f663888d82092e312c8aa13c9 Mon Sep 17 00:00:00 2001 From: Paul Stewart Date: Fri, 17 Sep 2010 16:09:02 -0700 Subject: [PATCH] Re-do argument passing to match zbehan's change zbehan's change: "run_remote_tests: destroy the with_spaces hack, use the --args instead of -a (which doesn't work)", despite apparently not working is in active use by others, both by folks using it on the command line and by the script cros_run_wifi_tests.sh. Fix the script to match the new argument. BUG=none TEST=rerun Review URL: http://codereview.chromium.org/3444010 --- bin/cros_run_wifi_tests.sh | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/bin/cros_run_wifi_tests.sh b/bin/cros_run_wifi_tests.sh index c11f3134ee..f948a4ef4c 100755 --- a/bin/cros_run_wifi_tests.sh +++ b/bin/cros_run_wifi_tests.sh @@ -63,7 +63,7 @@ append_flag () { append_arg () { local delim='' - [ -n "${run_remote_args}" ] && delim=',' + [ -n "${run_remote_args}" ] && delim=' ' run_remote_args="${run_remote_args}${delim}$*" } @@ -145,5 +145,6 @@ for arg in $*; do append_arg $arg done -eval "exec "${script_root}/run_remote_tests.sh" \ - -a "${run_remote_args}" --remote=${remote} $run_remote_flags $FLAGS_ARGV" +eval "exec ${script_root}/run_remote_tests.sh \ + --args=\"${run_remote_args}\" --remote=${remote} $run_remote_flags \ + $FLAGS_ARGV"