diff options
author | 2017-10-10 14:08:54 +0200 | |
---|---|---|
committer | 2017-10-11 10:00:00 +0200 | |
commit | 7c605cf6ea9755a06e5abb16a631faac8ebe2937 (patch) | |
tree | 58b2a1df3d14e9942c91729655375602c0198ead /src/main/java/com/google/devtools/build/lib/rules/java/proto/JavaLiteProtoAspect.java | |
parent | ca74482825e0c0ca5d119eceab74ba4292428557 (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.java | 4 |
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); } |