Filter out middleman artifacts in RunfilesSupplierImpl

--
MOS_MIGRATED_REVID=94477507

1 files changed, 3 insertions, 3 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/RunfilesSupplierImpl.java b/src/main/java/com/google/devtools/build/lib/analysis/RunfilesSupplierImpl.java
index ccdc89021d..0546988380 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/RunfilesSupplierImpl.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/RunfilesSupplierImpl.java
@@ -17,6 +17,7 @@ package com.google.devtools.build.lib.analysis;
 import com.google.common.annotations.VisibleForTesting;
 import com.google.common.collect.ImmutableMap;
 import com.google.common.collect.ImmutableSet;
+import com.google.common.collect.Iterables;
 import com.google.devtools.build.lib.actions.Artifact;
 import com.google.devtools.build.lib.actions.RunfilesSupplier;
 import com.google.devtools.build.lib.vfs.PathFragment;
@@ -51,9 +52,8 @@ public class RunfilesSupplierImpl implements RunfilesSupplier {
   public Iterable<Artifact> getArtifacts() {
     ImmutableSet.Builder<Artifact> builder = ImmutableSet.builder();
     for (Entry<PathFragment, Runfiles> entry : inputRunfiles.entrySet()) {
-      // TODO(bazel-team): We can likely do without middlemen here, but we should filter that at
-      // the Runfiles level.
-      builder.addAll(entry.getValue().getAllArtifacts());
+      builder.addAll(
+          Iterables.filter(entry.getValue().getAllArtifacts(), Artifact.MIDDLEMAN_FILTER));
     }
     return builder.build();
   }