[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH OSSTEST] standalone: Extend -h to support ident=host style specifications



Ian Campbell writes ("[PATCH OSSTEST] standalone: Extend -h to support 
ident=host style specifications"):
> Allowing for multi-host tests.
...
>       job=$1; shift
> -     ./cs-adjust-flight -v $flight runvar-del $job host
> +     ./cs-adjust-flight -v $flight runvar-del $job '/host$'

This regexp is slightly wrong because host runvars are all
  (?:\w+_)?host$

> +         hosts="$ident=$h $hosts"

Any reason you push it on the front rather than adding it to the end ?
It _shouldn't_ matter, but what if someone specifies -h foo=X -h foo=Y ?

Ian.

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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.