Fix string representation for the Root class

It can be accessed as ctx.genfiles_dir and should be printed as before for
compatibility.

PiperOrigin-RevId: 162360940

1 files changed, 8 insertions, 1 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/actions/Root.java b/src/main/java/com/google/devtools/build/lib/actions/Root.java
index afd5a62bcb..48abb2385e 100644
--- a/src/main/java/com/google/devtools/build/lib/actions/Root.java
+++ b/src/main/java/com/google/devtools/build/lib/actions/Root.java
@@ -20,6 +20,8 @@ import com.google.devtools.build.lib.concurrent.ThreadSafety.Immutable;
 import com.google.devtools.build.lib.skylarkinterface.SkylarkCallable;
 import com.google.devtools.build.lib.skylarkinterface.SkylarkModule;
 import com.google.devtools.build.lib.skylarkinterface.SkylarkModuleCategory;
+import com.google.devtools.build.lib.skylarkinterface.SkylarkPrinter;
+import com.google.devtools.build.lib.skylarkinterface.SkylarkValue;
 import com.google.devtools.build.lib.util.Preconditions;
 import com.google.devtools.build.lib.vfs.Path;
 import com.google.devtools.build.lib.vfs.PathFragment;
@@ -48,7 +50,7 @@ import javax.annotation.Nullable;
     doc = "A root for files. The roots are the directories containing files, and they are mapped "
         + "together into a single directory tree to form the execution environment.")
 @Immutable
-public final class Root implements Comparable<Root>, Serializable {
+public final class Root implements Comparable<Root>, Serializable, SkylarkValue {
 
   /**
    * Returns the given path as a source root. The path may not be {@code null}.
@@ -217,4 +219,9 @@ public final class Root implements Comparable<Root>, Serializable {
   public String toString() {
     return path + (isSourceRoot() ? "[source]" : "[derived]");
   }
+
+  @Override
+  public void repr(SkylarkPrinter printer) {
+    printer.append(toString());
+  }
 }