Include NODEP_LABEL in the set of attributes that are not reported as dependencies of a target (along with outputs and NODEP_LABEL_LIST, which was just special because of visibility).

--
MOS_MIGRATED_REVID=87180069

1 files changed, 3 insertions, 3 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/packages/AbstractAttributeMapper.java b/src/main/java/com/google/devtools/build/lib/packages/AbstractAttributeMapper.java
index 20b9304fc9..f03ee6bf08 100644
--- a/src/main/java/com/google/devtools/build/lib/packages/AbstractAttributeMapper.java
+++ b/src/main/java/com/google/devtools/build/lib/packages/AbstractAttributeMapper.java
@@ -134,10 +134,10 @@ public abstract class AbstractAttributeMapper implements AttributeMap {
     for (Attribute attribute : ruleClass.getAttributes()) {
       Type<?> type = attribute.getType();
       // TODO(bazel-team): This is incoherent: we shouldn't have to special-case these types
-      // for our visitation policy (e.g., why is Type.NODEP_LABEL_LIST excluded but not
-      // Type.NODEP_LABEL?). But this is the semantics the calling code requires. Audit
+      // for our visitation policy. But this is the semantics the calling code requires. Audit
       // exactly which calling code expects what and clean up this interface.
-      if (type == Type.OUTPUT || type == Type.OUTPUT_LIST || type == Type.NODEP_LABEL_LIST) {
+      if (type == Type.OUTPUT || type == Type.OUTPUT_LIST
+              || type == Type.NODEP_LABEL || type == Type.NODEP_LABEL_LIST) {
         continue;
       }
       for (Object value : visitAttribute(attribute.getName(), type)) {