[Piglit] [PATCH 1/6] Change --tests to --include-tests

[Dylan Baker]

That makes sense, I'll post a V2 soon.


On Mon, Mar 4, 2013 at 2:29 PM, Ken Phillis Jr <kphillisjr at gmail.com> wrote:

> As I stated in another email, It'll probably be a good idea to allow
> the deprecated behavior for a few more months, and instead print a
> warning message when users attempt to use deprecated options.
>
> On Sat, Mar 2, 2013 at 6:00 PM, Dylan Baker <baker.dylan.c at gmail.com>
> wrote:
> > Excluding tests already uses a similar syntax (--exclude-tests), so this
> > is more consistent.
> >
> > Signed-off-by: Dylan Baker <baker.dylan.c at gmail.com>
> > ---
> >  piglit-print-commands.py | 6 +++---
> >  piglit-run.py            | 6 +++---
> >  2 files changed, 6 insertions(+), 6 deletions(-)
> >
> > diff --git a/piglit-print-commands.py b/piglit-print-commands.py
> > index 7a3ba14..c01248b 100755
> > --- a/piglit-print-commands.py
> > +++ b/piglit-print-commands.py
> > @@ -48,7 +48,7 @@ Prints a list of all the tests and how to run them.
>  Ex:
> >
> >  Options:
> >    -h, --help                Show this message
> > -  -t regexp, --tests=regexp Run only matching tests (can be used more
> > +  -t regexp, --include-tests=regexp Run only matching tests (can be
> used more
> >                              than once)
> >    -x regexp, --exclude-tests=regexp Exclude matching tests (can be used
> >                              more than once)
> > @@ -70,7 +70,7 @@ def main():
> >         try:
> >                 option_list = [
> >                          "help",
> > -                        "tests=",
> > +                        "include-tests=",
> >                          "exclude-tests=",
> >                          ]
> >                 options, args = getopt(sys.argv[1:], "ht:x:",
> option_list)
> > @@ -85,7 +85,7 @@ def main():
> >         for name, value in options:
> >                 if name in ('-h', '--help'):
> >                         usage()
> > -               elif name in ('-t', '--tests'):
> > +               elif name in ('-t', '--include-tests'):
> >                         test_filter.append(value)
> >                         env.filter.append(re.compile(value))
> >                 elif name in ('-x', '--exclude-tests'):
> > diff --git a/piglit-run.py b/piglit-run.py
> > index 8da99a0..0ab8fc6 100755
> > --- a/piglit-run.py
> > +++ b/piglit-run.py
> > @@ -45,7 +45,7 @@ Usage: %(progName)s [options] [profile.tests] [results]
> >  Options:
> >    -h, --help                Show this message
> >    -d, --dry-run             Do not execute the tests
> > -  -t regexp, --tests=regexp Run only matching tests (can be used more
> > +  -t regexp, --include-tests=regexp Run only matching tests (can be
> used more
> >                              than once)
> >    -x regexp, --exclude-tests=regexp Excludey matching tests (can be used
> >                              more than once)
> > @@ -81,7 +81,7 @@ def main():
> >                          "dry-run",
> >                          "resume",
> >                          "valgrind",
> > -                        "tests=",
> > +                        "include-tests=",
> >                          "name=",
> >                          "exclude-tests=",
> >                          "concurrent=",
> > @@ -106,7 +106,7 @@ def main():
> >                         OptionResume = True
> >                 elif name in ('--valgrind'):
> >                         env.valgrind = True
> > -               elif name in ('-t', '--tests'):
> > +               elif name in ('-t', '--include-tests'):
> >                         test_filter.append(value)
> >                         env.filter.append(re.compile(value))
> >                 elif name in ('-x', '--exclude-tests'):
> > --
> > 1.8.1.2
> >
> > _______________________________________________
> > Piglit mailing list
> > Piglit at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/piglit
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20130305/ef861d39/attachment.html>