aboutsummaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorGravatar cushon <cushon@google.com>2017-04-17 18:11:20 +0200
committerGravatar Klaus Aehlig <aehlig@google.com>2017-04-18 11:31:25 +0200
commit70f5d58f0e3d581fbb2bbf821a21ef0b301e1f3c (patch)
tree09a2e7b464ff91855e86b4f60617553777063f98
parent4db5c96f3042c91bbc5724fe54164538e03804a2 (diff)
Report an error for missing java_library.exported_plugins
PiperOrigin-RevId: 153349925
-rw-r--r--src/main/java/com/google/devtools/build/lib/bazel/rules/java/BazelJavaLibraryRule.java14
1 files changed, 9 insertions, 5 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/bazel/rules/java/BazelJavaLibraryRule.java b/src/main/java/com/google/devtools/build/lib/bazel/rules/java/BazelJavaLibraryRule.java
index 240cc99982..6f1e81016c 100644
--- a/src/main/java/com/google/devtools/build/lib/bazel/rules/java/BazelJavaLibraryRule.java
+++ b/src/main/java/com/google/devtools/build/lib/bazel/rules/java/BazelJavaLibraryRule.java
@@ -112,9 +112,10 @@ public final class BazelJavaLibraryRule implements RuleDefinition {
<code>deps</code>.
</p>
<!-- #END_BLAZE_RULE.ATTRIBUTE --> */
- .add(attr("exports", LABEL_LIST)
- .allowedRuleClasses(BazelJavaRuleClasses.ALLOWED_RULES_IN_DEPS)
- .allowedFileTypes(/*May not have files in exports!*/))
+ .add(
+ attr("exports", LABEL_LIST)
+ .allowedRuleClasses(BazelJavaRuleClasses.ALLOWED_RULES_IN_DEPS)
+ .allowedFileTypes(/*May not have files in exports!*/ ))
/* <!-- #BLAZE_RULE(java_library).ATTRIBUTE(neverlink) -->
Whether this library should only be used for compilation and not at runtime.
@@ -145,8 +146,11 @@ public final class BazelJavaLibraryRule implements RuleDefinition {
labels in <code><a href="${link java_library.plugins}">plugins</a></code>.
</p>
<!-- #END_BLAZE_RULE.ATTRIBUTE --> */
- .add(attr("exported_plugins", LABEL_LIST).cfg(HOST).allowedRuleClasses("java_plugin")
- .legacyAllowAnyFileType())
+ .add(
+ attr("exported_plugins", LABEL_LIST)
+ .cfg(HOST)
+ .allowedRuleClasses("java_plugin")
+ .allowedFileTypes())
.advertiseProvider(JavaSourceInfoProvider.class)
.advertiseProvider(JavaCompilationArgsProvider.class)
.build();