Re-land "Update docs and command line flags for MB."

tools/mb/mb.py

Index: tools/mb/mb.py
diff --git a/tools/mb/mb.py b/tools/mb/mb.py
index e7ef57b63efe391f2cb8df238839cd744240b8ee..294e501c081b8372ac8a8c597968ada7740e00af 100755
--- a/tools/mb/mb.py
+++ b/tools/mb/mb.py
@@ -65,8 +65,8 @@ class MetaBuildWrapper(object):
                         help='Do a dry run (i.e., do nothing, just print '
                              'the commands that will run)')
       subp.add_argument('-q', '--quiet', action='store_true',
-                        help='Do not print anything, just return an exit '
-                             'code.')
+                        help='Do not print anything on success, '
+                             'just return an exit code.')
       subp.add_argument('-v', '--verbose', action='count',
                         help='verbose logging (may specify multiple times).')
 
@@ -108,7 +108,13 @@ class MetaBuildWrapper(object):
 
     subp = subps.add_parser('validate',
                             help='validate the config file')
-    AddCommonOptions(subp)
+    subp.add_argument('-f', '--config-file', metavar='PATH',
+                      default=self.default_config,
+                      help='path to config file '
+                          '(default is //tools/mb/mb_config.pyl)')

[M-A Ruel, 2015/07/17 01:17:13]

default is %default
?

[Dirk Pranke, 2015/07/17 01:22:48]

We don't want that because the actual default is an absolute path, not a
src-relative path.

[M-A Ruel, 2015/07/17 01:25:27]

ok!

+    subp.add_argument('-q', '--quiet', action='store_true',
+                      help='Do not print anything on success, '
+                           'just return an exit code.')
     subp.set_defaults(func=self.CmdValidate)
 
     subp = subps.add_parser('help',