Use an aspect to calculate the Android neverlink libraries.

This cuts the .java -> .android dependency between Java packages, which is nice at the cost of some cavalier over-estimation of the direct dependencies in AndroidNeverlinkAspect.

--
MOS_MIGRATED_REVID=94745544

11 files changed, 194 insertions, 35 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/packages/RuleClass.java b/src/main/java/com/google/devtools/build/lib/packages/RuleClass.java
index 93a0165d45..d841c2c991 100644
--- a/src/main/java/com/google/devtools/build/lib/packages/RuleClass.java
+++ b/src/main/java/com/google/devtools/build/lib/packages/RuleClass.java
@@ -470,6 +470,8 @@ public final class RuleClass {
               name);
           attributes.put(attrName, attribute);
         }
+
+        advertisedProviders.addAll(parent.getAdvertisedProviders());
       }
       // TODO(bazel-team): move this testonly attribute setting to somewhere else
       // preferably to some base RuleClass implementation.
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 f0a4778e00..2a36d8df5f 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
@@ -294,7 +294,6 @@ public abstract class AndroidBinary implements RuleConfiguredTargetFactory {
         deployJar,
         filesBuilder,
         resourceApk,
-        applicationManifest,
         ruleContext.getPrerequisiteArtifacts(PROGUARD_SPECS, Mode.TARGET).list(),
         proguardMapping,
         tools);
@@ -629,12 +628,11 @@ public abstract class AndroidBinary implements RuleConfiguredTargetFactory {
   }
 
   /** Applies the proguard specifications, and creates a ProguardedJar. */
-  public static ProguardOutput applyProguard(RuleContext ruleContext,
+  private static ProguardOutput applyProguard(RuleContext ruleContext,
       AndroidCommon common,
       Artifact deployJarArtifact,
       NestedSetBuilder<Artifact> filesBuilder,
       ResourceApk resourceApk,
-      ApplicationManifest applicationManifest,
       ImmutableList<Artifact> proguardSpecs,
       Artifact proguardMapping,
       AndroidTools tools) {
@@ -658,8 +656,9 @@ public abstract class AndroidBinary implements RuleConfiguredTargetFactory {
         proguardSpecs, proguardMapping, tools.getAndroidJar(), proguardOutputJar, filesBuilder);
   }
 
-  public static ProguardOutput createProguardAction(RuleContext ruleContext, AndroidCommon common,
-      FilesToRunProvider proguard, Artifact jar, ImmutableList<Artifact> proguardSpecs,
+  private static ProguardOutput createProguardAction(RuleContext ruleContext, AndroidCommon common,
+      FilesToRunProvider proguard,
+      Artifact jar, ImmutableList<Artifact> proguardSpecs,
       Artifact proguardMapping, Artifact androidJar, Artifact proguardOutputJar,
       NestedSetBuilder<Artifact> filesBuilder) {
     Iterable<Artifact> libraryJars = NestedSetBuilder.<Artifact>naiveLinkOrder()
diff --git a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidCommon.java b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidCommon.java
index 08984b49ce..093a7fe9f3 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidCommon.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidCommon.java
@@ -23,6 +23,7 @@ import com.google.common.collect.ImmutableMap;
 import com.google.devtools.build.lib.actions.Artifact;
 import com.google.devtools.build.lib.actions.MiddlemanFactory;
 import com.google.devtools.build.lib.actions.ResourceSet;
+import com.google.devtools.build.lib.analysis.AnalysisUtils;
 import com.google.devtools.build.lib.analysis.FilesToRunProvider;
 import com.google.devtools.build.lib.analysis.OutputGroupProvider;
 import com.google.devtools.build.lib.analysis.RuleConfiguredTarget.Mode;
@@ -54,6 +55,7 @@ import com.google.devtools.build.lib.rules.java.JavaCompilationArgsProvider;
 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.JavaNativeLibraryProvider;
+import com.google.devtools.build.lib.rules.java.JavaRuntimeJarProvider;
 import com.google.devtools.build.lib.rules.java.JavaSemantics;
 import com.google.devtools.build.lib.rules.java.JavaSourceJarsProvider;
 import com.google.devtools.build.lib.rules.java.JavaTargetAttributes;
@@ -102,7 +104,7 @@ public class AndroidCommon {
   public AndroidCommon(RuleContext ruleContext, JavaCommon javaCommon) {
     this.ruleContext = ruleContext;
     this.javaCommon = javaCommon;
-    this.asNeverLink = this.javaCommon.isNeverLink();
+    this.asNeverLink = JavaCommon.isNeverLink(ruleContext);
   }
 
   /**
@@ -123,24 +125,31 @@ public class AndroidCommon {
 
   /**
    * Collects the transitive neverlink dependencies.
+   *
+   * @param ruleContext the context of the rule neverlink deps are to be computed for
+   * @param deps the targets to be treated as dependencies
+   * @param runtimeJars the runtime jars produced by the rule (non-transitive)
+   *
+   * @return a nested set of the neverlink deps.
    */
   public static NestedSet<Artifact> collectTransitiveNeverlinkLibraries(
-      RuleContext ruleContext, JavaCommon javaCommon) {
+      RuleContext ruleContext, Iterable<? extends TransitiveInfoCollection> deps,
+      ImmutableList<Artifact> runtimeJars) {
     NestedSetBuilder<Artifact> builder = NestedSetBuilder.naiveLinkOrder();
-    for (AndroidNeverLinkLibrariesProvider dep :
-        javaCommon.getDependencies(AndroidNeverLinkLibrariesProvider.class)) {
-      builder.addTransitive(dep.getTransitiveNeverLinkLibraries());
+
+    for (AndroidNeverLinkLibrariesProvider provider : AnalysisUtils.getProviders(deps,
+        AndroidNeverLinkLibrariesProvider.class)) {
+      builder.addTransitive(provider.getTransitiveNeverLinkLibraries());
     }
-    if (javaCommon.isNeverLink()) {
-      builder.addAll(javaCommon.getJavaCompilationArtifacts().getRuntimeJars());
-      for (TransitiveInfoCollection dep : javaCommon.getDependencies()) {
-        if (dep.getProvider(AndroidNeverLinkLibrariesProvider.class) != null) {
-          JavaCompilationArgsProvider javaArgs = dep.getProvider(JavaCompilationArgsProvider.class);
-          Preconditions.checkState(javaArgs != null, ruleContext.getLabel());
-          builder.addTransitive(javaArgs.getRecursiveJavaCompilationArgs().getRuntimeJars());
-        }
+
+    if (JavaCommon.isNeverLink(ruleContext)) {
+      builder.addAll(runtimeJars);
+      for (JavaCompilationArgsProvider provider : AnalysisUtils.getProviders(
+          deps, JavaCompilationArgsProvider.class)) {
+        builder.addTransitive(provider.getRecursiveJavaCompilationArgs().getRuntimeJars());
       }
     }
+
     return builder.build();
   }
 
@@ -419,7 +428,10 @@ public class AndroidCommon {
     javaCommon.setClassPathFragment(new ClasspathConfiguredFragment(
         javaCommon.getJavaCompilationArtifacts(), attributes, asNeverLink));
     
-    transitiveNeverlinkLibraries = collectTransitiveNeverlinkLibraries(ruleContext, javaCommon);
+    transitiveNeverlinkLibraries = collectTransitiveNeverlinkLibraries(
+        ruleContext,
+        javaCommon.getDependencies(),
+        javaCommon.getJavaCompilationArtifacts().getRuntimeJars());
     topLevelSourceJars = ImmutableList.of(srcJar);
     transitiveSourceJars = javaCommon.collectTransitiveSourceJars(srcJar);
 
@@ -446,6 +458,8 @@ public class AndroidCommon {
 
     return builder
         .setFilesToBuild(filesToBuild)
+        .add(JavaRuntimeJarProvider.class,
+            new JavaRuntimeJarProvider(javaCommon.getJavaCompilationArtifacts().getRuntimeJars()))
         .add(RunfilesProvider.class, RunfilesProvider.simple(runfiles))
         .add(AndroidResourcesProvider.class, new AndroidResourcesProvider(
             ruleContext.getLabel(), transitiveResources))
@@ -454,8 +468,6 @@ public class AndroidCommon {
             javaCompilationArgs, recursiveJavaCompilationArgs,
             compileTimeDependencyArtifacts,
             NestedSetBuilder.<Artifact>emptySet(Order.STABLE_ORDER)))
-        .add(AndroidNeverLinkLibrariesProvider.class, new AndroidNeverLinkLibrariesProvider(
-            transitiveNeverlinkLibraries))
         .addOutputGroup(OutputGroupProvider.HIDDEN_TOP_LEVEL,
             collectHiddenTopLevelArtifacts(ruleContext))
         .addOutputGroup(JavaSemantics.SOURCE_JARS_OUTPUT_GROUP, transitiveSourceJars);
diff --git a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidLibraryBaseRule.java b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidLibraryBaseRule.java
index 0d3ebb8fd1..06e9b0eb6d 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidLibraryBaseRule.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidLibraryBaseRule.java
@@ -29,6 +29,7 @@ import com.google.devtools.build.lib.packages.RuleClass.Builder.RuleClassType;
 import com.google.devtools.build.lib.rules.android.AndroidRuleClasses.AndroidAaptBaseRule;
 import com.google.devtools.build.lib.rules.android.AndroidRuleClasses.AndroidBaseRule;
 import com.google.devtools.build.lib.rules.android.AndroidRuleClasses.AndroidResourceSupportRule;
+import com.google.devtools.build.lib.rules.java.JavaCompilationArgsProvider;
 import com.google.devtools.build.lib.rules.java.JavaSemantics;
 
 /**
@@ -156,6 +157,7 @@ public final class AndroidLibraryBaseRule implements RuleDefinition {
         }))
         .add(attr("$android_manifest_merge_tool", LABEL).cfg(HOST).exec().value(env.getLabel(
             AndroidRuleClasses.MANIFEST_MERGE_TOOL_LABEL)))
+        .advertiseProvider(JavaCompilationArgsProvider.class)
         .build();
   }
 
diff --git a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidNeverlinkAspect.java b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidNeverlinkAspect.java
new file mode 100644
index 0000000000..189b6fb736
--- /dev/null
+++ b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidNeverlinkAspect.java
@@ -0,0 +1,84 @@
+// Copyright 2015 Google Inc. All rights reserved.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//    http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+package com.google.devtools.build.lib.rules.android;
+
+import com.google.common.collect.ImmutableList;
+import com.google.devtools.build.lib.analysis.Aspect;
+import com.google.devtools.build.lib.analysis.ConfiguredAspectFactory;
+import com.google.devtools.build.lib.analysis.ConfiguredTarget;
+import com.google.devtools.build.lib.analysis.RuleConfiguredTarget.Mode;
+import com.google.devtools.build.lib.analysis.RuleContext;
+import com.google.devtools.build.lib.analysis.TransitiveInfoCollection;
+import com.google.devtools.build.lib.packages.AspectDefinition;
+import com.google.devtools.build.lib.packages.Type;
+import com.google.devtools.build.lib.rules.java.JavaCommon;
+import com.google.devtools.build.lib.rules.java.JavaCompilationArgsProvider;
+import com.google.devtools.build.lib.rules.java.JavaRuntimeJarProvider;
+
+import java.util.ArrayList;
+import java.util.List;
+
+/**
+ * An aspect that collects neverlink libraries in the transitive closure.
+ *
+ * <p>Used for determining the -libraryjars argument of the Proguard invocation in Android binaries.
+ *
+ * <p>One would think that using the compile time classpath would be enough, but alas, those are
+ * ijars,
+ */
+public class AndroidNeverlinkAspect implements ConfiguredAspectFactory {
+  private static final ImmutableList<String> ATTRIBUTES = ImmutableList.of(
+      "deps", "exports", "runtime_deps", "binary_under_test", "$instrumentation_test_runner");
+
+  @Override
+  public Aspect create(ConfiguredTarget base, RuleContext ruleContext) {
+    if (!JavaCommon.getConstraints(ruleContext).contains("android")
+        && !ruleContext.getRule().getRuleClass().startsWith("android_")) {
+      return new Aspect.Builder().build();
+    }
+
+    List<TransitiveInfoCollection> deps = new ArrayList<>();
+
+    // This is probably an overestimate, but it's fine -- Proguard doesn't care if there are more
+    // jars in -libraryjars than is required. The alternative would be somehow getting
+    // JavaCommon.getDependencies() here, which would be fugly.
+    for (String attribute : ATTRIBUTES) {
+      if (!ruleContext.getRule().getRuleClassObject().hasAttr(attribute, Type.LABEL_LIST)) {
+        continue;
+      }
+
+      deps.addAll(ruleContext.getPrerequisites(attribute, Mode.TARGET));
+    }
+
+    return new Aspect.Builder()
+        .addProvider(AndroidNeverLinkLibrariesProvider.class,
+            new AndroidNeverLinkLibrariesProvider(AndroidCommon.collectTransitiveNeverlinkLibraries(
+                ruleContext,
+                deps,
+                base.getProvider(JavaRuntimeJarProvider.class).getRuntimeJars())))
+        .build();
+  }
+
+  @Override
+  public AspectDefinition getDefinition() {
+    AspectDefinition.Builder builder = new AspectDefinition.Builder("AndroidNeverlinkAspect");
+    for (String attribute : ATTRIBUTES) {
+      builder.attributeAspect(attribute, AndroidNeverlinkAspect.class);
+    }
+
+    return builder
+        .requireProvider(JavaCompilationArgsProvider.class)
+        .build();
+  }
+}
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 6664288934..f859fc47e7 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
@@ -45,6 +45,7 @@ import com.google.devtools.build.lib.packages.RuleClass.Builder;
 import com.google.devtools.build.lib.packages.RuleClass.Builder.RuleClassType;
 import com.google.devtools.build.lib.packages.TriState;
 import com.google.devtools.build.lib.packages.Type;
+import com.google.devtools.build.lib.rules.java.JavaCompilationArgsProvider;
 import com.google.devtools.build.lib.rules.java.JavaSemantics;
 import com.google.devtools.build.lib.syntax.Label;
 import com.google.devtools.build.lib.util.FileType;
@@ -551,8 +552,11 @@ public final class AndroidRuleClasses {
           <code>cc_library</code> wrapping or producing <code>.so</code> native libraries for the
           Android target platform.
           <!-- #END_BLAZE_RULE.ATTRIBUTE --> */
-          .override(builder.copy("deps").cfg(ANDROID_SPLIT_TRANSITION)
-              .allowedRuleClasses(ALLOWED_DEPENDENCIES).allowedFileTypes())
+          .override(builder.copy("deps")
+              .cfg(ANDROID_SPLIT_TRANSITION)
+              .allowedRuleClasses(ALLOWED_DEPENDENCIES)
+              .allowedFileTypes()
+              .aspect(AndroidNeverlinkAspect.class))
           // Proguard rule specifying master list of classes to keep during legacy multidexing.
           .add(attr("$build_incremental_dexmanifest", LABEL).cfg(HOST).exec()
               .value(env.getLabel(AndroidRuleClasses.BUILD_INCREMENTAL_DEXMANIFEST_LABEL)))
@@ -664,6 +668,7 @@ com/google/common/base/Objects.class
             </ul>
           <!-- #END_BLAZE_RULE.ATTRIBUTE --> */
           .add(attr("legacy_native_support", TRISTATE).value(TriState.AUTO))
+          .advertiseProvider(JavaCompilationArgsProvider.class)
           .build();
       }
 
diff --git a/src/main/java/com/google/devtools/build/lib/rules/java/JavaCommon.java b/src/main/java/com/google/devtools/build/lib/rules/java/JavaCommon.java
index 66dafc2dda..ef72a40b49 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/java/JavaCommon.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/java/JavaCommon.java
@@ -159,6 +159,12 @@ public class JavaCommon {
         .build();
   }
 
+  public static ImmutableList<String> getConstraints(RuleContext ruleContext) {
+    return ruleContext.getRule().isAttrDefined("constraints", Type.STRING_LIST)
+        ? ImmutableList.copyOf(ruleContext.attributes().get("constraints", Type.STRING_LIST))
+        : ImmutableList.<String>of();
+  }
+
   public void setClassPathFragment(ClasspathConfiguredFragment classpathFragment) {
     this.classpathFragment = classpathFragment;
   }
@@ -634,7 +640,7 @@ public class JavaCommon {
    *
    * @return the value of the neverlink attribute.
    */
-  public final boolean isNeverLink() {
+  public static final boolean isNeverLink(RuleContext ruleContext) {
     return ruleContext.getRule().isAttrDefined("neverlink", Type.BOOLEAN) &&
         ruleContext.attributes().get("neverlink", Type.BOOLEAN);
   }
diff --git a/src/main/java/com/google/devtools/build/lib/rules/java/JavaImport.java b/src/main/java/com/google/devtools/build/lib/rules/java/JavaImport.java
index 5e59311081..f82eefb843 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/java/JavaImport.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/java/JavaImport.java
@@ -74,11 +74,11 @@ public class JavaImport implements RuleConfiguredTargetFactory {
     CppCompilationContext transitiveCppDeps = common.collectTransitiveCppDeps();
     NestedSet<LinkerInput> transitiveJavaNativeLibraries =
         common.collectTransitiveJavaNativeLibraries();
-
+    boolean neverLink = JavaCommon.isNeverLink(ruleContext);
     JavaCompilationArgs javaCompilationArgs = common.collectJavaCompilationArgs(
-        false, common.isNeverLink(), compilationArgsFromSources());
+        false, neverLink, compilationArgsFromSources());
     JavaCompilationArgs recursiveJavaCompilationArgs = common.collectJavaCompilationArgs(
-        true, common.isNeverLink(), compilationArgsFromSources());
+        true, neverLink, compilationArgsFromSources());
     NestedSet<Artifact> transitiveJavaSourceJars =
         collectTransitiveJavaSourceJars(ruleContext, srcJar);
     if (srcJar != null) {
@@ -87,7 +87,7 @@ public class JavaImport implements RuleConfiguredTargetFactory {
 
     // The "neverlink" attribute is transitive, so if it is enabled, we don't add any
     // runfiles from this target or its dependencies.
-    Runfiles runfiles = common.isNeverLink() ?
+    Runfiles runfiles = neverLink ?
         Runfiles.EMPTY :
         new Runfiles.Builder()
             // add the jars to the runfiles
@@ -120,7 +120,9 @@ public class JavaImport implements RuleConfiguredTargetFactory {
     common.addTransitiveInfoProviders(ruleBuilder, filesToBuild, null);
     return ruleBuilder
         .setFilesToBuild(filesToBuild)
-        .add(JavaNeverlinkInfoProvider.class, new JavaNeverlinkInfoProvider(common.isNeverLink()))
+        .add(JavaRuntimeJarProvider.class,
+            new JavaRuntimeJarProvider(common.getJavaCompilationArtifacts().getRuntimeJars()))
+        .add(JavaNeverlinkInfoProvider.class, new JavaNeverlinkInfoProvider(neverLink))
         .add(RunfilesProvider.class, RunfilesProvider.simple(runfiles))
         .add(CcLinkParamsProvider.class, new CcLinkParamsProvider(ccLinkParamsStore))
         .add(JavaCompilationArgsProvider.class, new JavaCompilationArgsProvider(
diff --git a/src/main/java/com/google/devtools/build/lib/rules/java/JavaImportBaseRule.java b/src/main/java/com/google/devtools/build/lib/rules/java/JavaImportBaseRule.java
index 5cbfe13dad..8db95eb81e 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/java/JavaImportBaseRule.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/java/JavaImportBaseRule.java
@@ -71,6 +71,7 @@ public class JavaImportBaseRule implements RuleDefinition {
         .add(attr("constraints", STRING_LIST)
             .orderIndependent()
             .nonconfigurable("used in Attribute.validityPredicate implementations (loading time)"))
+        .advertiseProvider(JavaCompilationArgsProvider.class)
         .build();
   }
 
diff --git a/src/main/java/com/google/devtools/build/lib/rules/java/JavaLibrary.java b/src/main/java/com/google/devtools/build/lib/rules/java/JavaLibrary.java
index 8df9711d04..68f5ccd942 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/java/JavaLibrary.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/java/JavaLibrary.java
@@ -65,7 +65,8 @@ public class JavaLibrary implements RuleConfiguredTargetFactory {
     JavaCompilationHelper helper = new JavaCompilationHelper(
         ruleContext, semantics, common.getJavacOpts(), attributesBuilder);
     helper.addLibrariesToAttributes(common.targetsTreatedAsDeps(ClasspathType.COMPILE_ONLY));
-    helper.addProvidersToAttributes(common.compilationArgsFromSources(), common.isNeverLink());
+    helper.addProvidersToAttributes(common.compilationArgsFromSources(),
+        JavaCommon.isNeverLink(ruleContext));
 
     if (ruleContext.hasErrors()) {
       return null;
@@ -131,18 +132,19 @@ public class JavaLibrary implements RuleConfiguredTargetFactory {
           javaArtifactsBuilder);
     }
 
+    boolean neverLink = JavaCommon.isNeverLink(ruleContext);
     common.setJavaCompilationArtifacts(javaArtifactsBuilder.build());
     common.setClassPathFragment(new ClasspathConfiguredFragment(
-        common.getJavaCompilationArtifacts(), attributes, common.isNeverLink()));
+        common.getJavaCompilationArtifacts(), attributes, neverLink));
     CppCompilationContext transitiveCppDeps = common.collectTransitiveCppDeps();
 
     NestedSet<Artifact> transitiveSourceJars = common.collectTransitiveSourceJars(srcJar);
 
     // If sources are empty, treat this library as a forwarding node for dependencies.
     JavaCompilationArgs javaCompilationArgs = common.collectJavaCompilationArgs(
-        false, common.isNeverLink(), common.compilationArgsFromSources());
+        false, neverLink, common.compilationArgsFromSources());
     JavaCompilationArgs recursiveJavaCompilationArgs = common.collectJavaCompilationArgs(
-        true, common.isNeverLink(), common.compilationArgsFromSources());
+        true, neverLink, common.compilationArgsFromSources());
     NestedSet<Artifact> compileTimeJavaDepArtifacts = common.collectCompileTimeDependencyArtifacts(
         common.getJavaCompilationArtifacts().getCompileTimeDependencyArtifact());
     NestedSet<Artifact> runTimeJavaDepArtifacts = NestedSetBuilder.emptySet(Order.STABLE_ORDER);
@@ -175,7 +177,7 @@ public class JavaLibrary implements RuleConfiguredTargetFactory {
     // The "neverlink" attribute is transitive, so we don't add any
     // runfiles from this target or its dependencies.
     Runfiles runfiles = Runfiles.EMPTY;
-    if (!common.isNeverLink()) {
+    if (!neverLink) {
       Runfiles.Builder runfilesBuilder = new Runfiles.Builder().addArtifacts(
           common.getJavaCompilationArtifacts().getRuntimeJars());
 
@@ -217,9 +219,11 @@ public class JavaLibrary implements RuleConfiguredTargetFactory {
     common.addTransitiveInfoProviders(builder, filesToBuild, classJar);
 
     builder
+        .add(JavaRuntimeJarProvider.class,
+            new JavaRuntimeJarProvider(common.getJavaCompilationArtifacts().getRuntimeJars()))
         .add(RunfilesProvider.class, RunfilesProvider.simple(runfiles))
         .setFilesToBuild(filesToBuild)
-        .add(JavaNeverlinkInfoProvider.class, new JavaNeverlinkInfoProvider(common.isNeverLink()))
+        .add(JavaNeverlinkInfoProvider.class, new JavaNeverlinkInfoProvider(neverLink))
         .add(CppCompilationContext.class, transitiveCppDeps)
         .add(JavaCompilationArgsProvider.class, new JavaCompilationArgsProvider(
             javaCompilationArgs, recursiveJavaCompilationArgs,
diff --git a/src/main/java/com/google/devtools/build/lib/rules/java/JavaRuntimeJarProvider.java b/src/main/java/com/google/devtools/build/lib/rules/java/JavaRuntimeJarProvider.java
new file mode 100644
index 0000000000..7cbdca0953
--- /dev/null
+++ b/src/main/java/com/google/devtools/build/lib/rules/java/JavaRuntimeJarProvider.java
@@ -0,0 +1,42 @@
+// Copyright 2015 Google Inc. All rights reserved.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//    http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+package com.google.devtools.build.lib.rules.java;
+
+import com.google.common.collect.ImmutableList;
+import com.google.devtools.build.lib.actions.Artifact;
+import com.google.devtools.build.lib.analysis.TransitiveInfoProvider;
+import com.google.devtools.build.lib.concurrent.ThreadSafety.Immutable;
+
+/**
+ * This {@link com.google.devtools.build.lib.analysis.TransitiveInfoProvider} contains the .jar
+ * files to be put on the runtime classpath by the configured target.
+ *
+ * <p>Unlike {@link com.google.devtools.build.lib.rules.java.JavaCompilationArgs#getRuntimeJars()},
+ * it does not contain transitive runtime jars, only those produced by the configured target itself.
+ *
+ * <p>The reason why this class exists is that neverlink libraries do not contain the compiled jar
+ * in {@link com.google.devtools.build.lib.rules.java.JavaCompilationArgs#getRuntimeJars()}.
+ */
+@Immutable
+public final class JavaRuntimeJarProvider implements TransitiveInfoProvider {
+  private final ImmutableList<Artifact> runtimeJars;
+
+  public JavaRuntimeJarProvider(ImmutableList<Artifact> runtimeJars) {
+    this.runtimeJars = runtimeJars;
+  }
+
+  public ImmutableList<Artifact> getRuntimeJars() {
+    return runtimeJars;
+  }
+}