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

Dylan Baker baker.dylan.c at gmail.com
Sat Mar 2 16:00:51 PST 2013


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



More information about the Piglit mailing list