Fix command output

The first line lacked the subcommand like:

$ gbp pull --help
Usage: gbp [options] - safely update a repository from remote

instead of

$ gbp pull --help
Usage: gbp pull [options] - safely update a repository from remote
           ^^^^

diff --git a/gbp/config.py b/gbp/config.py
index fa6e6798d074c25a289c82e06063d3591d62d196..9ac1ee8f538bba0a087e7317100c0330fadd03c4 100644 (file)
--- a/gbp/config.py
+++ b/gbp/config.py
@@ -20,6 +20,7 @@ from optparse import OptionParser, OptionGroup, Option, OptionValueError
 from ConfigParser import SafeConfigParser, NoSectionError
 from copy import copy
 import os.path
+import sys
 try:
     from gbp.version import gbp_version
 except ImportError:
@@ -356,7 +357,7 @@ class GbpOptionParser(OptionParser):
         else:
             self.config['filter'] = []
 
-    def __init__(self, command, prefix='', usage=None, sections=[]):
+    def __init__(self, command=None, prefix='', usage=None, sections=[]):
         """
         @param command: the command to build the config parser for
         @type command: C{str}
@@ -368,6 +369,10 @@ class GbpOptionParser(OptionParser):
             to parse
         @type sections: C{list} of C{str}
         """
+        if not command:
+            command = os.path.basename(sys.argv[0])
+            if command == 'gbp':
+                command += " " + os.path.basename(sys.argv[1])
         self.command = command
         self.sections = sections
         self.prefix = prefix
@@ -375,6 +380,7 @@ class GbpOptionParser(OptionParser):
         self.config_files = self.get_config_files()
         self._parse_config_files()
         OptionParser.__init__(self, option_class=GbpOption,
+                              prog=command,
                               usage=usage, version='%s %s' % (self.command,
                                                               gbp_version))
 

diff --git a/gbp/scripts/buildpackage.py b/gbp/scripts/buildpackage.py
index 753ad64aaaf59090d40b476d1993b4424884d96c..0b14b3f67133639c72b9d286b68c32eb750d0916 100755 (executable)
--- a/gbp/scripts/buildpackage.py
+++ b/gbp/scripts/buildpackage.py
@@ -375,7 +375,7 @@ def parse_args(argv, prefix):
             args.append(arg)
 
     try:
-        parser = GbpOptionParserDebian(command=os.path.basename(argv[0]), prefix=prefix)
+        parser = GbpOptionParserDebian(prefix=prefix)
     except ConfigParser.ParsingError as err:
         gbp.log.err(err)
         return None, None, None

diff --git a/gbp/scripts/clone.py b/gbp/scripts/clone.py
index 251cef21dd24cb03442bf013a26d3395494becb4..1ddee3fba91a52142501484d961c3252baf506bc 100755 (executable)
--- a/gbp/scripts/clone.py
+++ b/gbp/scripts/clone.py
@@ -31,8 +31,7 @@ import gbp.log
 
 def parse_args (argv):
     try:
-        parser = GbpOptionParser(command=os.path.basename(argv[0]), prefix='',
-                                 usage='%prog [options] repository - clone a remote repository')
+        parser = GbpOptionParser(usage='%prog [options] repository - clone a remote repository')
     except ConfigParser.ParsingError as err:
         gbp.log.err(err)
         return None, None

diff --git a/gbp/scripts/create_remote_repo.py b/gbp/scripts/create_remote_repo.py
index c8c4a36add969f77755d3b2b7120d00ea06f2a1e..95a10b0040b6c3808f1521801fbb3a512cb910a2 100644 (file)
--- a/gbp/scripts/create_remote_repo.py
+++ b/gbp/scripts/create_remote_repo.py
@@ -245,8 +245,7 @@ def parse_args(argv, sections=[]):
     else:
         sections = []
 
-    parser = GbpOptionParserDebian(command=os.path.basename(argv[0]), prefix='',
-                                   usage='%prog [options] - '
+    parser = GbpOptionParserDebian(usage='%prog [options] - '
                                    'create a remote repository',
                                    sections=sections)
     branch_group = GbpOptionGroup(parser,

diff --git a/gbp/scripts/dch.py b/gbp/scripts/dch.py
index a848d6d2cda70aac7bb9497380e55cc3fe13fdb6..30c90d162bd1c768b2209a16eda54dbb76796582 100644 (file)
--- a/gbp/scripts/dch.py
+++ b/gbp/scripts/dch.py
@@ -292,8 +292,7 @@ def main(argv):
     branch = None
 
     try:
-        parser = GbpOptionParserDebian(command=os.path.basename(argv[0]), prefix='',
-                                       usage='%prog [options] paths')
+        parser = GbpOptionParserDebian(usage='%prog [options] paths')
     except ConfigParser.ParsingError as err:
         gbp.log.err(err)
         return 1

diff --git a/gbp/scripts/import_dsc.py b/gbp/scripts/import_dsc.py
index 630422bfe4048fb885b8dd4176b3595590e6f8d2..cbc67f0fd4c48984467d4202d6761382aec26363 100644 (file)
--- a/gbp/scripts/import_dsc.py
+++ b/gbp/scripts/import_dsc.py
@@ -207,8 +207,7 @@ def set_bare_repo_options(options):
 
 def parse_args(argv):
     try:
-        parser = GbpOptionParserDebian(command=os.path.basename(argv[0]), prefix='',
-                                       usage='%prog [options] /path/to/package.dsc')
+        parser = GbpOptionParserDebian(usage='%prog [options] /path/to/package.dsc')
     except ConfigParser.ParsingError as err:
         gbp.log.err(err)
         return None, None

diff --git a/gbp/scripts/import_orig.py b/gbp/scripts/import_orig.py
index aae93fabb739f49bcfa55685a1a476c7fa612b4a..968aadb1fa5a46116a013d4fc44608fde61abbb4 100644 (file)
--- a/gbp/scripts/import_orig.py
+++ b/gbp/scripts/import_orig.py
@@ -183,8 +183,7 @@ def set_bare_repo_options(options):
 
 def parse_args(argv):
     try:
-        parser = GbpOptionParserDebian(command=os.path.basename(argv[0]), prefix='',
-                                       usage='%prog [options] /path/to/upstream-version.tar.gz | --uscan')
+        parser = GbpOptionParserDebian(usage='%prog [options] /path/to/upstream-version.tar.gz | --uscan')
     except ConfigParser.ParsingError as err:
         gbp.log.err(err)
         return None, None

diff --git a/gbp/scripts/pq.py b/gbp/scripts/pq.py
index 41d3ddf82da4e3d68de0668ad1763c131348084f..5e941262df862322cc79e577ab73262e5730ec1d 100755 (executable)
--- a/gbp/scripts/pq.py
+++ b/gbp/scripts/pq.py
@@ -212,8 +212,7 @@ def switch_pq(repo, current):
 
 def parse_args(argv):
     try:
-        parser = GbpOptionParserDebian(command=os.path.basename(argv[0]), prefix='',
-                                   usage="%prog [options] action - maintain patches on a patch queue branch\n"
+        parser = GbpOptionParserDebian(usage="%prog [options] action - maintain patches on a patch queue branch\n"
         "Actions:\n"
         "  export         export the patch queue associated to the current branch\n"
         "                 into a quilt patch series in debian/patches/ and update the\n"

diff --git a/gbp/scripts/pull.py b/gbp/scripts/pull.py
index 03be9fc7fd03b8a51845e32dc02ae64864065a05..6cc47d4c8ea78710944416b6d56cb8ca00947819 100755 (executable)
--- a/gbp/scripts/pull.py
+++ b/gbp/scripts/pull.py
@@ -70,8 +70,7 @@ def fast_forward_branch(branch, repo, options):
 
 def parse_args(argv):
     try:
-        parser = GbpOptionParser(command=os.path.basename(argv[0]), prefix='',
-                             usage='%prog [options] - safely update a repository from remote')
+        parser = GbpOptionParser(usage='%prog [options] - safely update a repository from remote')
     except ConfigParser.ParsingError as err:
         gbp.log.err(err)
         return None, None