From 2aa1a98d892e75401b27cb7a57db529f025cc645 Mon Sep 17 00:00:00 2001
From: Dmitry Lomov <dslomov@google.com>
Date: Tue, 20 Oct 2015 12:18:36 +0000
Subject: Improve error diagnostics for Skylark aspects.

--
MOS_MIGRATED_REVID=105851371
---
 .../google/devtools/build/lib/skyframe/AspectValue.java    | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

(limited to 'src/main/java/com/google/devtools/build/lib/skyframe/AspectValue.java')

diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/AspectValue.java b/src/main/java/com/google/devtools/build/lib/skyframe/AspectValue.java
index 5a4052fb8c..884d5dbc72 100644
--- a/src/main/java/com/google/devtools/build/lib/skyframe/AspectValue.java
+++ b/src/main/java/com/google/devtools/build/lib/skyframe/AspectValue.java
@@ -56,6 +56,8 @@ public final class AspectValue extends ActionLookupValue {
 
     public abstract AspectParameters getParameters();
 
+    public abstract String getDescription();
+
     public BuildConfiguration getConfiguration() {
       return configuration;
     }
@@ -87,6 +89,11 @@ public final class AspectValue extends ActionLookupValue {
       return aspect.getParameters();
     }
 
+    @Override
+    public String getDescription() {
+      return String.format("%s of %s", aspect.getAspectFactory().getName(), getLabel());
+    }
+
     @Override
     SkyFunctionName getType() {
       return SkyFunctions.NATIVE_ASPECT;
@@ -151,6 +158,13 @@ public final class AspectValue extends ActionLookupValue {
       return AspectParameters.EMPTY;
     }
 
+    @Override
+    public String getDescription() {
+      // Skylark aspects are referred to on command line with <file>%<value name>
+      return String.format(
+          "%s%%%s of %s", extensionFile.toString(), skylarkFunctionName, getLabel());
+    }
+
     @Override
     SkyFunctionName getType() {
       return SkyFunctions.SKYLARK_ASPECT;
-- 
cgit v1.2.3