[Xen-devel] [PATCH OSSTEST 4/5] standalone: Make it possible to pass options to run-test

[Ian Campbell <ian.campbell@xxxxxxxxxx>]

Currently the remainder of the comnand line is passed after the host=
ident, which allows for other idents to be given, which isn't all that
useful in practice.

Instead arrange that any additional options up to a "--" marker are
passed before host= and anything after are passed after.

Since the options themselves have a leading -- this can confuse the
scripts own option parsing, meaning you may need more than one "--"
marker, the first to separate the standalone helper args from the ts
args and a second to separate from any ident optiopns.

./standalone run-test -h $HOST -- test-amd64-amd64-xl-xsm ts-host-install 
--rescue -- guest=debian

Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
---
v3: Use an array var for correct whitespace handling.
---
 standalone | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/standalone b/standalone
index 3408d24..71f656e 100755
--- a/standalone
+++ b/standalone
@@ -318,11 +318,18 @@ case $op in
        job=$1; shift
        ts=$1; shift
 
+       options=()
+       for i in $@ ; do
+           if [ x$i = x-- ] ; then shift; break ; fi
+           options+=("$i")
+           shift
+       done
+
        OSSTEST_CONFIG=$config \
        OSSTEST_FLIGHT=$flight \
        OSSTEST_HOST_REUSE=$reuse \
        OSSTEST_JOB=$job \
-           with_logging logs/$flight/$job.$ts.log ./$ts $hosts $@
+           with_logging logs/$flight/$job.$ts.log ./$ts ${options[@]} $hosts $@
        ;;
 
     get-job-status)
-- 
2.5.3


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel