Change in vdsm[master]: vdsm-tool: Add command name to args for command

dkuznets at redhat.com dkuznets at redhat.com
Wed May 14 11:52:56 UTC 2014


Dima Kuznetsov has posted comments on this change.

Change subject: vdsm-tool: Add command name to args for command
......................................................................


Patch Set 5:

(1 comment)

http://gerrit.ovirt.org/#/c/26575/5/vdsm-tool/vdsm-tool
File vdsm-tool/vdsm-tool:

Line 105: 
Line 106: 
Line 107: def print_command_usage(cmd, stream=sys.stdout):
Line 108:     print >> stream, 'usage:'
Line 109:     print >> stream, sys.argv[0], '[options]', cmd.__doc__.lstrip()
> I would very much prefer here to have forwards compatibility and cleaner sy
please consider http://gerrit.ovirt.org/#/c/27678/
Line 110: 
Line 111: 
Line 112: def _usage_command(cmd_name, cmd_docs):
Line 113:     print


-- 
To view, visit http://gerrit.ovirt.org/26575
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I831bde947f451805e086e25f1c0b9943b17e7bd8
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov <dkuznets at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Dima Kuznetsov <dkuznets at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Yaniv Eylon <yaniv.eylon at gmail.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: mooli tayer <mtayer at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list