From e749782d9a994d6db512f249da3bf7d184ea53d7 Mon Sep 17 00:00:00 2001 From: Stu Hood Date: Fri, 15 Jun 2018 20:01:43 -0400 Subject: [PATCH] Mark a few options that should not show up in `./pants help`. (#5968) ### Problem `./pants help` contains core options that are useful to every pants command, and the vast majority of global options are hidden in order to keep it concise. A few non-essential options ended up there recently. ### Solution Hide them. --- src/python/pants/option/global_options.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/python/pants/option/global_options.py b/src/python/pants/option/global_options.py index 250c6e2014b..0a5b4ee1771 100644 --- a/src/python/pants/option/global_options.py +++ b/src/python/pants/option/global_options.py @@ -196,6 +196,7 @@ def register_bootstrap_options(cls, register): register('--target-spec-file', type=list, dest='target_spec_files', daemon=False, help='Read additional specs from this file, one per line') register('--verify-config', type=bool, default=True, daemon=False, + advanced=True, help='Verify that all config file values correspond to known options.') register('--build-ignore', advanced=True, type=list, fromfile=True, @@ -212,6 +213,7 @@ def register_bootstrap_options(cls, register): register('--glob-expansion-failure', type=str, choices=GlobMatchErrorBehavior.allowed_values, default=GlobMatchErrorBehavior.default_option_value, + advanced=True, help="Raise an exception if any targets declaring source files " "fail to match any glob provided in the 'sources' argument.") @@ -291,7 +293,8 @@ def register_bootstrap_options(cls, register): # This option changes the parser behavior in a fundamental way (which currently invalidates # all caches), and needs to be parsed out early, so we make it a bootstrap option. register('--build-file-imports', choices=['allow', 'warn', 'error'], default='warn', - help='Whether to allow import statements in BUILD files') + advanced=True, + help='Whether to allow import statements in BUILD files') register('--remote-store-server', advanced=True, help='host:port of grpc server to use as remote execution file store.') @@ -310,6 +313,7 @@ def register_bootstrap_options(cls, register): # This should eventually deprecate the RunTracker worker count, which is used for legacy cache # lookups via CacheSetup in TaskBase. register('--process-execution-parallelism', type=int, default=multiprocessing.cpu_count(), + advanced=True, help='Number of concurrent processes that may be executed either locally and remotely.') @classmethod