perf/compare-logs.py | 36 +++++++++---------------------------
1 file changed, 9 insertions(+), 27 deletions(-)
New commits:
commit d6c3272ee0d8d0ba83f9591486b08e5263638ba7
Author: Xisco Fauli <xiscofa...@libreoffice.org>
AuthorDate: Wed Mar 9 16:00:38 2022 +0100
Commit: Xisco Fauli <xiscofa...@libreoffice.org>
CommitDate: Wed Mar 9 16:01:36 2022 +0100
perf: use argparse in compare-logs.py
Change-Id: If14a4705fcda5ffdac2a6e5915a7d8b5bf322905
diff --git a/perf/compare-logs.py b/perf/compare-logs.py
index 82e6d0c..90a60da 100755
--- a/perf/compare-logs.py
+++ b/perf/compare-logs.py
@@ -25,7 +25,8 @@
# in which case the provisions of the GPLv3+ or the LGPLv3+ are applicable
# instead of those above.
-import sys, os, getopt
+import sys, os
+import argparse
def readFile(url):
d = {}
@@ -37,35 +38,16 @@ def readFile(url):
d[key] = float(val)
return d
-def usage():
- message = """usage: {program} [option]..."
- -h | --help: print usage information
- --old=URL path to old file to be compared
- --new=URL path to new file to be comparted"""
-
- print(message.format(program = os.path.basename(sys.argv[0])))
-
-
if __name__ == "__main__":
- try:
- opts,args = getopt.getopt(sys.argv[1:], "o:n:h",
- ["old=", "new=", "help"])
- except getopt.GetoptError:
- usage()
- sys.exit(1)
+ parser = argparse.ArgumentParser()
+
+ parser.add_argument('--old', action='store', dest="old", required=True)
+ parser.add_argument('--new', action="store", dest="new", required=True)
- for opt, arg in opts:
- if opt in ('-h', '--help'):
- usage()
- sys.exit(1)
- elif opt in ('-o', '--old'):
- old_url = arg
- elif opt in ('-n', '--new'):
- new_url = arg
- else:
- usage()
- sys.exit(1)
+ results = parser.parse_args()
+ old_url = results.old
+ new_url = results.new
if not os.path.exists(old_url):
print("File " + old_url + " doesn't exist!")
