From c761661bfcc84eb2bdd229e10ecee338a53d3b41 Mon Sep 17 00:00:00 2001
From: Greg Estren <gregce@google.com>
Date: Thu, 21 Apr 2016 20:43:10 +0000
Subject: Add JavaSemantics.checkRule to Android rules.

--
MOS_MIGRATED_REVID=120480051
---
 .../com/google/devtools/build/lib/rules/android/AndroidBinary.java     | 1 +
 .../com/google/devtools/build/lib/rules/android/AndroidLibrary.java    | 1 +
 .../google/devtools/build/lib/rules/android/AndroidRuleClasses.java    | 3 ++-
 3 files changed, 4 insertions(+), 1 deletion(-)

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

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 2303e184b8..e534f372a8 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
@@ -96,6 +96,7 @@ public abstract class AndroidBinary implements RuleConfiguredTargetFactory {
         ruleContext.getPrerequisites("deps", Mode.TARGET));
     JavaCommon javaCommon = new JavaCommon(
         ruleContext, javaSemantics, deps, deps, deps);
+    javaSemantics.checkRule(ruleContext, javaCommon);
 
     AndroidCommon androidCommon = new AndroidCommon(
         javaCommon, true /* asNeverLink */, true /* exportDeps */);
diff --git a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidLibrary.java b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidLibrary.java
index 642cac823d..2e4e7756c4 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidLibrary.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidLibrary.java
@@ -65,6 +65,7 @@ public abstract class AndroidLibrary implements RuleConfiguredTargetFactory {
     NestedSet<Artifact> transitiveProguardConfigs =
         new ProguardLibrary(ruleContext).collectProguardSpecs();
     JavaCommon javaCommon = new JavaCommon(ruleContext, javaSemantics);
+    javaSemantics.checkRule(ruleContext, javaCommon);
     AndroidCommon androidCommon = new AndroidCommon(javaCommon);
 
     boolean definesLocalResources =
diff --git a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidRuleClasses.java b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidRuleClasses.java
index f93487feb9..56db345650 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidRuleClasses.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidRuleClasses.java
@@ -245,7 +245,8 @@ public final class AndroidRuleClasses {
       "cc_library",
       "java_import",
       "java_library",
-      "proto_library"};
+      "proto_library" // TODO(gregce): remove this line when no such dependencies exist
+  };
 
   public static final SafeImplicitOutputsFunction ANDROID_BINARY_IMPLICIT_OUTPUTS =
       new SafeImplicitOutputsFunction() {
-- 
cgit v1.2.3