commit 8bc997b9c2dbd00f444ce25687ff49c563f848c9
author Ryo Hashimoto <hashimoto@chromium.org> Wed Jan 22 18:46:17 2014 +0900
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Tue Jan 28 10:30:24 2014 +0000
tree 1a1228f98c31a13dfdbd836acb96c9d8f70bb059
parent 950c6cb6a9537c6dad979b7310c96bf75991033a

Show "Caching Options:" section when "cros chrome-sdk --help"

"cros chrome-sdk --help" doesn't show --cache-dir option despite the fact that it uses the option.
It also lists "--clear-sdk-cache" in "optional arguments" section and
it results in difficulty when trying to escape from error state caused by corrupted cache.

Move --clear-sdk-cache to "Caching Options" section.
Add use_caching_options to CrosCommand
Remove unused variable SUPPORTS_CACHING from BaseParser

BUG=None
TEST=cros chrome-sdk --help

Change-Id: I4f265fdfa3c7af11f3e8c12d3a154ccc92705076
Reviewed-on: https://chromium-review.googlesource.com/183299
Reviewed-by: David James <davidjames@chromium.org>
Tested-by: Ryo Hashimoto <hashimoto@chromium.org>
Commit-Queue: Ryo Hashimoto <hashimoto@chromium.org>

scripts/cros.py

diff --git a/scripts/cros.py b/scripts/cros.py
index 39af59c..45470a4 100644
--- a/scripts/cros.py
+++ b/scripts/cros.py
@@ -20,6 +20,7 @@
     epilog = getattr(class_def, 'EPILOG', None)
     sub_parser = subparsers.add_parser(
         cmd_name, description=class_def.__doc__, epilog=epilog,
+        caching=class_def.use_caching_options,
         formatter_class=commandline.argparse.RawDescriptionHelpFormatter)
     class_def.AddParser(sub_parser)