diff options
author | 2018-07-31 08:53:47 -0700 | |
---|---|---|
committer | 2018-07-31 08:54:56 -0700 | |
commit | 04d8a923ffd81b35eca52e34accbab930571d626 (patch) | |
tree | 91ea58897f9e97626be0e2964f4d88099a4d628a /src/main/java/com/google/devtools/build/lib/rules/java | |
parent | 504a6d20845c65a346e6f376611aa7fdafb65950 (diff) |
Remove output jar from runfiles and the ijar from the OutputJar when sources
are not provided.
RELNOTES: none.
PiperOrigin-RevId: 206768891
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/java/JavaInfoBuildHelper.java | 7 | ||||
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/java/JavaLibraryHelper.java | 7 |
2 files changed, 11 insertions, 3 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/java/JavaInfoBuildHelper.java b/src/main/java/com/google/devtools/build/lib/rules/java/JavaInfoBuildHelper.java index fed792eb5f..4ff6468940 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/java/JavaInfoBuildHelper.java +++ b/src/main/java/com/google/devtools/build/lib/rules/java/JavaInfoBuildHelper.java @@ -536,6 +536,12 @@ final class JavaInfoBuildHelper { sourceJarsP -> transitiveSourceJars.addTransitive(sourceJarsP.getTransitiveSourceJars())); + // When sources are not provided, the subsequent output Jar will be empty. As such, the output + // Jar is omitted from the set of Runtime Jars. + if (!sourceJars.isEmpty() || !sourceFiles.isEmpty()) { + javaInfoBuilder.setRuntimeJars(ImmutableList.of(outputJar)); + } + return javaInfoBuilder .addProvider(JavaCompilationArgsProvider.class, javaCompilationArgsProvider) .addProvider( @@ -545,7 +551,6 @@ final class JavaInfoBuildHelper { .addProvider(JavaRunfilesProvider.class, new JavaRunfilesProvider(runfiles)) .addProvider(JavaPluginInfoProvider.class, transitivePluginsProvider) .setNeverlink(neverlink) - .setRuntimeJars(ImmutableList.of(outputJar)) .build(); } diff --git a/src/main/java/com/google/devtools/build/lib/rules/java/JavaLibraryHelper.java b/src/main/java/com/google/devtools/build/lib/rules/java/JavaLibraryHelper.java index c035f5c433..37929cb8b9 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/java/JavaLibraryHelper.java +++ b/src/main/java/com/google/devtools/build/lib/rules/java/JavaLibraryHelper.java @@ -267,8 +267,11 @@ public final class JavaLibraryHelper { /* instrumentationMetadataJar= */ null, nativeHeaderOutput); - artifactsBuilder.addRuntimeJar(output); - Artifact iJar = helper.createCompileTimeJarAction(output, artifactsBuilder); + Artifact iJar = null; + if (!sourceJars.isEmpty() || !sourceFiles.isEmpty()) { + artifactsBuilder.addRuntimeJar(output); + iJar = helper.createCompileTimeJarAction(output, artifactsBuilder); + } if (createOutputSourceJar) { helper.createSourceJarAction( |