Allow Skylark import lookup values to be computed inline in order to avoid reifying them in the Skyframe graph.

--
MOS_MIGRATED_REVID=103758591

1 files changed, 7 insertions, 1 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/FileSymlinkCycleUniquenessFunction.java b/src/main/java/com/google/devtools/build/lib/skyframe/FileSymlinkCycleUniquenessFunction.java
index 3ce5532b17..4077c65600 100644
--- a/src/main/java/com/google/devtools/build/lib/skyframe/FileSymlinkCycleUniquenessFunction.java
+++ b/src/main/java/com/google/devtools/build/lib/skyframe/FileSymlinkCycleUniquenessFunction.java
@@ -13,18 +13,24 @@
 // limitations under the License.
 package com.google.devtools.build.lib.skyframe;
 
+import com.google.devtools.build.lib.vfs.RootedPath;
 import com.google.devtools.build.skyframe.SkyFunction;
 import com.google.devtools.build.skyframe.SkyValue;
 
 /** A {@link SkyFunction} that has the side effect of reporting a file symlink cycle. */
 public class FileSymlinkCycleUniquenessFunction
-    extends AbstractFileSymlinkExceptionUniquenessFunction {
+    extends AbstractChainUniquenessFunction<RootedPath> {
   @Override
   protected SkyValue getDummyValue() {
     return FileSymlinkCycleUniquenessValue.INSTANCE;
   }
 
   @Override
+  protected String elementToString(RootedPath elt) {
+    return elt.asPath().toString();
+  }
+
+  @Override
   protected String getConciseDescription() {
     return "circular symlinks";
   }