From 6a5669a9b00ed4113ee3945c4a74c35c17a70728 Mon Sep 17 00:00:00 2001
From: ajmichael <ajmichael@google.com>
Date: Fri, 1 Dec 2017 13:25:19 -0800
Subject: Make AndroidSdkProvider.verifyPresence throw.

All call sites were just returning null if it failed anyways.

RELNOTES: None
PiperOrigin-RevId: 177632366
---
 .../com/google/devtools/build/lib/rules/android/AndroidBinary.java    | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

(limited to 'src/main/java/com/google/devtools/build/lib/rules/android/AndroidBinary.java')

diff --git a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidBinary.java b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidBinary.java
index d806620b12..b3044d3f0e 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidBinary.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidBinary.java
@@ -101,9 +101,7 @@ public abstract class AndroidBinary implements RuleConfiguredTargetFactory {
     CppSemantics cppSemantics = createCppSemantics();
     JavaSemantics javaSemantics = createJavaSemantics();
     AndroidSemantics androidSemantics = createAndroidSemantics();
-    if (!AndroidSdkProvider.verifyPresence(ruleContext)) {
-      return null;
-    }
+    AndroidSdkProvider.verifyPresence(ruleContext);
 
     NestedSetBuilder<Artifact> filesBuilder = NestedSetBuilder.stableOrder();
     JavaCommon javaCommon =
-- 
cgit v1.2.3