list-fixed-bugs: use argparse for the commandline

This makes the interface more friendly to users.

diff --git a/ChangeLog b/ChangeLog
index 5135c0178570fe0a91f4636831933a77c6e33fa2..183c55b1595627fd17697ebcdeaff070e78fe430 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2015-12-29  Mike Frysinger  <vapier@gentoo.org>
+
+       * scripts/list-fixed-bugs.py: Import argparse.  Call main instead.
+       (get_parser): New function.
+       (main): New function.
+
 2015-12-29  Rob Wu  <rob@robwu.nl>
 
        [BZ #19369]

diff --git a/scripts/list-fixed-bugs.py b/scripts/list-fixed-bugs.py
index 37e9a43cc13b52a103b1dac887b23a8fe4ba649e..0a61adcbabe53110a8d73e9fa27288fe0f51975d 100755 (executable)
--- a/scripts/list-fixed-bugs.py
+++ b/scripts/list-fixed-bugs.py
@@ -23,11 +23,21 @@ bugs marked as FIXED with that milestone, to be added to the NEWS file
 just before release.  The output is in UTF-8.
 """
 
+import argparse
 import json
 import sys
 import textwrap
 import urllib.request
 
+
+def get_parser():
+    """Return an argument parser for this module."""
+    parser = argparse.ArgumentParser(description=__doc__)
+    parser.add_argument('version',
+                        help='Release version to look up')
+    return parser
+
+
 def list_fixed_bugs(version):
     """List the bugs fixed in a given version."""
     url = ('https://sourceware.org/bugzilla/rest.cgi/bug?product=glibc'
@@ -42,5 +52,13 @@ def list_fixed_bugs(version):
                              subsequent_indent='    ') + '\n'
         sys.stdout.buffer.write(desc.encode('utf-8'))
 
+
+def main(argv):
+    """The main entry point."""
+    parser = get_parser()
+    opts = parser.parse_args(argv)
+    list_fixed_bugs(opts.version)
+
+
 if __name__ == '__main__':
-    list_fixed_bugs(sys.argv[1])
+    main(sys.argv[1:])