diff options
| author |  Klaus Aehlig <aehlig@google.com> | 2016-03-22 14:36:02 +0000 |
|---|---|---|
| committer |  Damien Martin-Guillerez <dmarting@google.com> | 2016-03-23 12:18:18 +0000 |
| commit | dc0fbb42ab22ab8f3205b0884069e1ffe03677c9 (patch) | |
| tree | 8a50b334948c61faf4aa2a254b1f3ca2a5135a4f /src/main/java/com/google/devtools/build/lib | |
| parent | ebd386280db58e1f303cbd07533a03c30820da28 (diff) | |
Make specific options win over common ones
When specifying which command names to parse options for, given the
command annotation, ensure that "common" comes first, so that it can
be overridden by more specific options.
--
Change-Id: I20cd90c098e993580c32297d1e36c8a6c939873a
Reviewed-on: https://bazel-review.googlesource.com/#/c/3091
MOS_MIGRATED_REVID=117821257
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib')
| -rw-r--r-- | src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandDispatcher.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandDispatcher.java b/src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandDispatcher.java index a426f47fd5..38021d518a 100644 --- a/src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandDispatcher.java +++ b/src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandDispatcher.java @@ -489,7 +489,7 @@ public class BlazeCommandDispatcher { List<String> result = new ArrayList<>(); getCommandNamesToParseHelper(commandAnnotation, result); result.add("common"); - // TODO(bazel-team): This statement is a NO-OP: Lists.reverse(result); + result = Lists.reverse(result); return result; } |