diff options
author | cushon <cushon@google.com> | 2018-03-16 12:57:52 -0700 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-03-16 12:59:22 -0700 |
commit | 7d46249768662cbc76545ae2ce09c26c3b976831 (patch) | |
tree | ac20e860a039d56c3dfd54ec42671cf1d8cd95aa /src/java_tools/buildjar/java/com | |
parent | 6873cd46c5a72de3620b36c418f1697115fc4492 (diff) |
Clean up after b/72379900
PiperOrigin-RevId: 189380767
Diffstat (limited to 'src/java_tools/buildjar/java/com')
-rw-r--r-- | src/java_tools/buildjar/java/com/google/devtools/build/buildjar/OptionsParser.java | 30 |
1 files changed, 0 insertions, 30 deletions
diff --git a/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/OptionsParser.java b/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/OptionsParser.java index 5c1856ba54..22906195a1 100644 --- a/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/OptionsParser.java +++ b/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/OptionsParser.java @@ -98,9 +98,6 @@ public final class OptionsParser { * @throws InvalidCommandLineException on an invalid option being passed. */ private void processCommandlineArgs(Deque<String> argQueue) throws InvalidCommandLineException { - boolean foundNewDependencyArgument = false; - boolean foundLegacyDependencyArgument = false; - for (String arg = argQueue.pollFirst(); arg != null; arg = argQueue.pollFirst()) { switch (arg) { case "--javacopts": @@ -109,25 +106,7 @@ public final class OptionsParser { break; case "--direct_dependencies": collectFlagArguments(directJars, argQueue, "--"); - foundNewDependencyArgument = true; break; - case "--direct_dependency": - { - // TODO(b/72379900): Remove this - String jar = getArgument(argQueue, arg); - getArgument(argQueue, arg); - directJars.add(jar); - foundLegacyDependencyArgument = true; - break; - } - case "--indirect_dependency": - { - // TODO(b/72379900): Remove this - getArgument(argQueue, arg); - getArgument(argQueue, arg); - foundLegacyDependencyArgument = true; - break; - } case "--strict_java_deps": strictJavaDeps = getArgument(argQueue, arg); break; @@ -213,15 +192,6 @@ public final class OptionsParser { throw new InvalidCommandLineException("unknown option : '" + arg + "'"); } } - // TODO(b/72379900): Remove the legacy part of this code - if (foundLegacyDependencyArgument && foundNewDependencyArgument) { - throw new InvalidCommandLineException( - "Found both new-style and old-style dependency arguments: " - + "Cannot use arguments from both " - + "(--direct_dependencies) and " - + "(--direct_dependency, --indirect_dependency) " - + "at the same time."); - } } private void sourcePathFromJavacOpts() { |