Commit e71ce8d8 authored by Andrew Grieve's avatar Andrew Grieve Committed by Commit Bot

Android: Make --fast-local-dev no longer imply -v

-v is not always desired, and does not play with with --local-output

Change-Id: Ia8bb3132f90e6da64953463fed523d97470802c9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1624828
Auto-Submit: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: default avatarPeter Wen <wnwen@chromium.org>
Reviewed-by: default avatarPeter Conn <peconn@chromium.org>
Reviewed-by: default avatarTed Choc <tedchoc@chromium.org>
Commit-Queue: Peter Wen <wnwen@chromium.org>
Commit-Queue: Peter Conn <peconn@chromium.org>
Commit-Queue: Ted Choc <tedchoc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#662218}
parent 28912898
...@@ -175,7 +175,6 @@ def AddCommonOptions(parser): ...@@ -175,7 +175,6 @@ def AddCommonOptions(parser):
class FastLocalDevAction(argparse.Action): class FastLocalDevAction(argparse.Action):
def __call__(self, parser, namespace, values, option_string=None): def __call__(self, parser, namespace, values, option_string=None):
namespace.verbose_count = max(namespace.verbose_count, 1)
namespace.num_retries = 0 namespace.num_retries = 0
namespace.enable_device_cache = True namespace.enable_device_cache = True
namespace.enable_concurrent_adb = True namespace.enable_concurrent_adb = True
...@@ -184,10 +183,12 @@ def AddCommonOptions(parser): ...@@ -184,10 +183,12 @@ def AddCommonOptions(parser):
parser.add_argument( parser.add_argument(
'--fast-local-dev', '--fast-local-dev',
type=bool, nargs=0, action=FastLocalDevAction, type=bool,
help='Alias for: --verbose --num-retries=0 ' nargs=0,
'--enable-device-cache --enable-concurrent-adb ' action=FastLocalDevAction,
'--skip-clear-data --extract-test-list-from-filter') help='Alias for: --num-retries=0 --enable-device-cache '
'--enable-concurrent-adb --skip-clear-data '
'--extract-test-list-from-filter')
# TODO(jbudorick): Remove this once downstream bots have switched to # TODO(jbudorick): Remove this once downstream bots have switched to
# api.test_results. # api.test_results.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment