931: add a -v option to turn debug = True in post-review.py

macdev%n*********@gtempacc******** (Google Code) (Is this you? Claim this profile.)
March 5, 2009 
It would be much appreciated to have a '-v' option in post-review.py, which
would during option parsing set the global value debug to be True, to
display debug info in a verbose way during review post.
chipx86
#1 chipx86 
You mean our -d (--debug) flag? We already have this.
  • +NeedInfo
#2 macdev%n*********@gtempacc******** (Google Code) (Is this you? Claim this profile.) 
I mean '-v' for verbose, which is (as I understand it) somewhat of a standard for
getting more verbose output from command line applications.

In fact, as a user (only one of ???) didn't even think to search for '-d' or
'--debug' in the help info. I just assumed such an option would be under '-v' or
'--verbose', or didn't exist.

Perhaps adding an alais of '-v' to do the same thing would make it easier for some
users who expect that behavior.
chipx86
#3 chipx86 
-v is also often an alias for --version, which I considered adding recently.

I'll be reorganizing the help output soon to group these by section (to match
upcoming docs), so maybe this will be easier to find, but I don't think that we want
to either remove -d (as so many people are familiar with it now) or give it an alias
(as that can lead to confusion between the two, and we already have too many flags as
it is).

The type of output that -d covers really is debugging information. I think if there's
anything useful that post-review should output that's not intended for debugging, we
should just output it. Keeps it simple and keeps the user informed.

So I'm going to close this, but with the intention of cleaning up help output to make
-d easier to find and to add a section on this in the new docs on debug output.
  • -NeedInfo
    +WontFix