diff --git a/src/main/java/com/google/devtools/build/lib/rules/java/NativeLibraryNestedSetBuilder.java b/src/main/java/com/google/devtools/build/lib/rules/java/NativeLibraryNestedSetBuilder.java index 7705ae853f8932..474180a55046e8 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/java/NativeLibraryNestedSetBuilder.java +++ b/src/main/java/com/google/devtools/build/lib/rules/java/NativeLibraryNestedSetBuilder.java @@ -25,7 +25,6 @@ import com.google.devtools.build.lib.rules.cpp.CppFileTypes; import com.google.devtools.build.lib.rules.cpp.LibraryToLink; import com.google.devtools.build.lib.util.FileType; -import java.util.List; /** A builder that helps construct nested sets of native libraries. */ public final class NativeLibraryNestedSetBuilder { @@ -97,16 +96,13 @@ private void addCcTarget(TransitiveInfoCollection dep) { /** Include files and genrule artifacts. */ private void addTarget(TransitiveInfoCollection dep) { - List soLibraries = + if (ruleContext.getFragment(JavaConfiguration.class).dontCollectSoArtifacts()) { + return; + } + for (Artifact artifact : FileType.filterList( dep.getProvider(FileProvider.class).getFilesToBuild().toList(), - CppFileTypes.SHARED_LIBRARY); - if (!soLibraries.isEmpty() - && ruleContext.getFragment(JavaConfiguration.class).dontCollectSoArtifacts()) { - ruleContext.ruleError( - ".so libraries as artifact (from filegroup or genrule) are present in the dependencies"); - } - for (Artifact artifact : soLibraries) { + CppFileTypes.SHARED_LIBRARY)) { builder.add( LibraryToLink.builder() .setLibraryIdentifier(CcLinkingOutputs.libraryIdentifierOf(artifact))