[MERGE] Verbosity levels and standard options
mbp at sourcefrog.net
Fri Aug 31 09:15:47 BST 2007
Here's a partial review:
That description sounds good.
I really like custom_help(); now we can give better help for some
things like -r which are a bit mysterious at present.
+ 'Options that are applied before the command is processed')
This is technically accurate but may not be very helpful to the user.
Would 'options for debugging bazaar' be better, or 'options that
control how bazaar runs'?
@@ -240,9 +240,8 @@
These options may be used with any command, and may appear in front of any
-command. (e.g. "bzr --quiet help").
+command. (e.g. "bzr --profile help").
---quiet Suppress informational output; only print errors and warnings.
--version Print the version number.
--no-aliases Do not process command aliases when running this command.
@@ -258,6 +257,7 @@
-Dhpss Trace smart protocol requests and responses.
-Dindex Trace major index operations.
-Dlock Trace when lockdir locks are taken or released.
--profile Profile execution using the hotshot profiler.
--lsprof Profile execution using the lsprof profiler.
--lsprof-file Profile execution using the lsprof profiler, and write the
Could you please group the -D options separate from the other
options, it looks a bit messy.
+Unlike global options, standard options can be used in aliases and
+may have more explicit help on a per command basis.
I think the second line is not relevant to users.
At the moment we have the somewhat odd situation that some things are
done by modifying global state in the trace module and some are passed
to the commands. It looks like this clears it up a bit, but I wonder
if we would do better to just pass every thing to the commands and get
rid of the globals altogether?
More information about the bazaar