aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/main/java/com/google/devtools/build/lib/rules/java/proto/JavaLiteProtoAspect.java
diff options
context:
space:
mode:
authorGravatar elenairina <elenairina@google.com>2017-10-10 14:08:54 +0200
committerGravatar Marcel Hlopko <hlopko@google.com>2017-10-11 10:00:00 +0200
commit7c605cf6ea9755a06e5abb16a631faac8ebe2937 (patch)
tree58b2a1df3d14e9942c91729655375602c0198ead /src/main/java/com/google/devtools/build/lib/rules/java/proto/JavaLiteProtoAspect.java
parentca74482825e0c0ca5d119eceab74ba4292428557 (diff)
Return JavaInfo providers only once in Java rules.
Previously the java rules returned some providers twice: once as regular providers and once wrapped in JavaInfo (e.g. JavaCompilationArgsProvider). This is unnecessary, inefficient and error prone. JavaInfo should be the only way of returning these providers. RELNOTES: None. PiperOrigin-RevId: 171663550
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/java/proto/JavaLiteProtoAspect.java')
-rw-r--r--src/main/java/com/google/devtools/build/lib/rules/java/proto/JavaLiteProtoAspect.java4
1 files changed, 3 insertions, 1 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/java/proto/JavaLiteProtoAspect.java b/src/main/java/com/google/devtools/build/lib/rules/java/proto/JavaLiteProtoAspect.java
index cc6cc166a3..7d87f9891d 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/java/proto/JavaLiteProtoAspect.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/java/proto/JavaLiteProtoAspect.java
@@ -51,6 +51,7 @@ import com.google.devtools.build.lib.rules.java.JavaCompilationArtifacts;
import com.google.devtools.build.lib.rules.java.JavaCompilationHelper;
import com.google.devtools.build.lib.rules.java.JavaConfiguration;
import com.google.devtools.build.lib.rules.java.JavaHelper;
+import com.google.devtools.build.lib.rules.java.JavaInfo;
import com.google.devtools.build.lib.rules.java.JavaLibraryHelper;
import com.google.devtools.build.lib.rules.java.JavaRuleOutputJarsProvider;
import com.google.devtools.build.lib.rules.java.JavaSemantics;
@@ -256,7 +257,8 @@ public class JavaLiteProtoAspect extends NativeAspectClass implements Configured
.setJavacOpts(ProtoJavacOpts.constructJavacOpts(ruleContext));
helper.addDep(dependencyCompilationArgs).setCompilationStrictDepsMode(StrictDepsMode.OFF);
for (TransitiveInfoCollection t : getProtoRuntimeDeps()) {
- JavaCompilationArgsProvider provider = t.getProvider(JavaCompilationArgsProvider.class);
+ JavaCompilationArgsProvider provider =
+ JavaInfo.getProvider(JavaCompilationArgsProvider.class, t);
if (provider != null) {
helper.addDep(provider);
}