aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/java_tools/buildjar
diff options
context:
space:
mode:
authorGravatar cushon <cushon@google.com>2018-03-16 12:57:52 -0700
committerGravatar Copybara-Service <copybara-piper@google.com>2018-03-16 12:59:22 -0700
commit7d46249768662cbc76545ae2ce09c26c3b976831 (patch)
treeac20e860a039d56c3dfd54ec42671cf1d8cd95aa /src/java_tools/buildjar
parent6873cd46c5a72de3620b36c418f1697115fc4492 (diff)
Clean up after b/72379900
PiperOrigin-RevId: 189380767
Diffstat (limited to 'src/java_tools/buildjar')
-rw-r--r--src/java_tools/buildjar/java/com/google/devtools/build/buildjar/OptionsParser.java30
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() {