From d168a2eef36c7b2f1f6de41929acae1f9c1c638a Mon Sep 17 00:00:00 2001 From: Dylan Baker Date: Thu, 21 Mar 2013 09:20:40 -0700 Subject: piglit-summary-junit.py: Convert optparse to argparse Optparse has been deprecated in favor of argparse upstream (PEP 389 http://www.python.org/dev/peps/pep-0389/), and since we are using argparse already in the other modules, it makes sense to replace optparse and use only one parsing method in all piglit. V2: Dropped some unrelated code per Ken Graunke Signed-off-by: Dylan Baker Reviewed-by: Kenneth Graunke --- piglit-summary-junit.py | 32 ++++++++++++++++---------------- 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/piglit-summary-junit.py b/piglit-summary-junit.py index 65208889..7ed5de3d 100755 --- a/piglit-summary-junit.py +++ b/piglit-summary-junit.py @@ -24,7 +24,7 @@ # SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. -import optparse +import argparse import os import sys @@ -109,21 +109,21 @@ class Writer: def main(): - optparser = optparse.OptionParser( - usage="\n\t%prog [options] test.results", - version="%%prog") - optparser.add_option( - '-o', '--output', metavar='FILE', - type="string", dest="output", default='piglit.xml', - help="output filename") - (options, args) = optparser.parse_args(sys.argv[1:]) - - if len(args) != 1: - optparser.error('need to specify one test result') - usage() - - writer = Writer(options.output) - writer.write(args[0]) + parser = argparse.ArgumentParser() + parser.add_argument("-o", "--output", + metavar = "", + action = "store", + dest = "output", + default = "piglit.xml", + help = "Output filename") + parser.add_argument("testResults", + metavar = "", + help = "JSON results file to be converted") + args = parser.parse_args() + + + writer = Writer(args.output) + writer.write(args.testResults) if __name__ == "__main__": -- cgit v1.2.1