Sorry to nit pick, but could we change the option name to "--jobs" and
the abbreviated option to "-j".  This is consistent with how make does
it.

Acked-by: Ethan Jackson <et...@nicira.com>


On Tue, Jun 3, 2014 at 5:41 PM, Joe Stringer <joestrin...@nicira.com> wrote:
> Signed-off-by: Joe Stringer <joestrin...@nicira.com>
> ---
> v3: Set the default to -j8.
> v2: Make it configurable.
> v1: First post.
> ---
>  utilities/ovs-dev.py |    6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/utilities/ovs-dev.py b/utilities/ovs-dev.py
> index 2c3af19..fa74d57 100755
> --- a/utilities/ovs-dev.py
> +++ b/utilities/ovs-dev.py
> @@ -145,6 +145,10 @@ commands.append(make)
>
>  def check():
>      flags = ""
> +    if options.parallel:
> +        flags += "-j%d " % options.parallel
> +    else:
> +        flags += "-j8 "
>      if options.tests:
>          for arg in str.split(options.tests):
>              if arg[0].isdigit():
> @@ -364,6 +368,8 @@ def main():
>      parser.add_option_group(group)
>
>      group = optparse.OptionGroup(parser, "check")
> +    group.add_option("-p", dest="parallel", metavar="N_TESTS", type="int",
> +                     help="Run N_TESTS in parallel")
>      group.add_option("--tests", dest="tests", metavar="FILTER",
>                       help="""run specific tests and/or a test category
>                            eg, --tests=\"1-10 megaflow\"""")
> --
> 1.7.10.4
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to