diff options
author | cnsun <cnsun@google.com> | 2018-05-24 11:18:40 -0700 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-05-24 11:20:26 -0700 |
commit | 9ba727786ce6ce6c5b9b6d8790eae8c3bfbf3066 (patch) | |
tree | 86a0b4be2872fb07792c8a22537f989d205f47f0 /src/java_tools/import_deps_checker/java/com | |
parent | 0b5dd249f3749cf35e829a27341a50db478ec1e1 (diff) |
Remove the flag --fail_on_errors.
RELNOTES:none.
PiperOrigin-RevId: 197923013
Diffstat (limited to 'src/java_tools/import_deps_checker/java/com')
-rw-r--r-- | src/java_tools/import_deps_checker/java/com/google/devtools/build/importdeps/Main.java | 17 |
1 files changed, 1 insertions, 16 deletions
diff --git a/src/java_tools/import_deps_checker/java/com/google/devtools/build/importdeps/Main.java b/src/java_tools/import_deps_checker/java/com/google/devtools/build/importdeps/Main.java index 7c7c034c7e..73e71b5f1a 100644 --- a/src/java_tools/import_deps_checker/java/com/google/devtools/build/importdeps/Main.java +++ b/src/java_tools/import_deps_checker/java/com/google/devtools/build/importdeps/Main.java @@ -132,20 +132,9 @@ public class Main { help = "The rule label of the current target under analysis.") public String ruleLabel; - // TODO (cnsun): remove this flag. - @Option( - name = "fail_on_errors", - defaultValue = "false", - documentationCategory = OptionDocumentationCategory.UNDOCUMENTED, - effectTags = {OptionEffectTag.UNKNOWN}, - deprecationWarning = "This will be replaced with --checking_mode.", - help = "Fail on incomplete dependencies, otherwise emit warnings." - ) - public boolean failOnErrors; - @Option( name = "checking_mode", - defaultValue = "null", + defaultValue = "WARNING", documentationCategory = OptionDocumentationCategory.UNDOCUMENTED, effectTags = {OptionEffectTag.UNKNOWN}, converter = CheckingModeConverter.class, @@ -228,10 +217,6 @@ public class Main { options.jdepsOutput == null || !Files.isDirectory(options.jdepsOutput), "Invalid value of --jdeps_output: '%s'", options.jdepsOutput); - if (options.checkingMode == null) { - // Convert --fail_on_errors to --checking_mode - options.checkingMode = options.failOnErrors ? CheckingMode.ERROR : CheckingMode.WARNING; - } if (!options.fullClasspath.containsAll(options.directClasspath)) { ArrayList<Path> missing = Lists.newArrayList(options.directClasspath); missing.removeAll(options.fullClasspath); |