diff options
author | Michael Staib <mstaib@google.com> | 2017-03-17 23:15:16 +0000 |
---|---|---|
committer | Yue Gan <yueg@google.com> | 2017-03-20 11:43:06 +0000 |
commit | a5aa916dc8c0e99c8fc91893246381f72b7066a9 (patch) | |
tree | 4a8c20854fd779b12d554319677c48a240dbe918 /src/main/java/com/google/devtools/build/lib/rules/apple | |
parent | 42d313fa526398b76aa122106adf1518a784e12f (diff) |
Create "internal" category of command-line options.
This is intended to be used for "flags" which should never appear on the
command line - things like configuration distinguishers, which are used
internally and must be part of the build options, but should always be set
to their default value at the top level.
This is already a convention within Bazel, but doesn't actually work the way
Bazel expects - flags with spaces can be set by simply escaping or quoting
the spaces so that word splitting will not break on them. This means they
can also be matched by config_settings, which pass a single string.
Forbidding the parser from matching these flags solves both of these
unintended cases.
Existing cases like this have also been converted to internal.
--
PiperOrigin-RevId: 150497246
MOS_MIGRATED_REVID=150497246
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/apple')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/apple/AppleCommandLineOptions.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/apple/AppleCommandLineOptions.java b/src/main/java/com/google/devtools/build/lib/rules/apple/AppleCommandLineOptions.java index e79688092e..fb42de007e 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/apple/AppleCommandLineOptions.java +++ b/src/main/java/com/google/devtools/build/lib/rules/apple/AppleCommandLineOptions.java @@ -188,7 +188,7 @@ public class AppleCommandLineOptions extends FragmentOptions { @Option(name = "apple configuration distinguisher", defaultValue = "UNKNOWN", converter = ConfigurationDistinguisherConverter.class, - category = "undocumented") + category = "internal") public ConfigurationDistinguisher configurationDistinguisher; @Option(name = "ios_multi_cpus", |