From 44f2275c69c2b4f3637d888f82ff0b5cc232e4bc Mon Sep 17 00:00:00 2001 From: Lin Hua Cheng Date: Fri, 19 Apr 2013 03:20:57 -0700 Subject: [PATCH] Fix regression on running manage command. Options on the manage command (./run_tests.sh) is getting shuffled, this is due to the logic how testopts and testargs are extracted. For example, the command "./run_tests.sh -m startdashboard test --target test" is being translated into "startdashboard --target test test". The options after the "-m" should be passed as is as argument to the manage command. Fixes bug 1164505. Change-Id: Id1a0699050390014487459757b3bda9c7fcca405 --- run_tests.sh | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/run_tests.sh b/run_tests.sh index 6dac0d3513..f2e7e51758 100755 --- a/run_tests.sh +++ b/run_tests.sh @@ -80,6 +80,12 @@ manage=0 [ "$JOB_NAME" ] || JOB_NAME="default" function process_option { + # If running manage command, treat the rest of options as arguments. + if [ $manage -eq 1 ]; then + testargs="$testargs $1" + return 0 + fi + case "$1" in -h|--help) usage;; -V|--virtual-env) always_venv=1; never_venv=0;;