Use runfiles path, not execroot path, for Skylark's short_path

Will help with fixing #1233.

--
MOS_MIGRATED_REVID=121993946

1 files changed, 12 insertions, 8 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/actions/Artifact.java b/src/main/java/com/google/devtools/build/lib/actions/Artifact.java
index 3c71f88baf..e801a7e0a0 100644
--- a/src/main/java/com/google/devtools/build/lib/actions/Artifact.java
+++ b/src/main/java/com/google/devtools/build/lib/actions/Artifact.java
@@ -512,6 +512,18 @@ public class Artifact
     return relativePath;
   }
 
+  @SkylarkCallable(
+      name = "short_path",
+      structField = true,
+      doc =
+          "The path of this file relative to its root. This excludes the aforementioned "
+              + "<i>root</i>, i.e. configuration-specific fragments of the path. This is also the "
+              + "path under which the file is mapped if it's in the runfiles of a binary."
+  )
+  public final String getRunfilesPathString() {
+    return getRunfilesPath().getPathString();
+  }
+
   /**
    * Returns this.getExecPath().getPathString().
    */
@@ -541,14 +553,6 @@ public class Artifact
     return ShellUtils.shellEscape(getExecPathString());
   }
 
-  @SkylarkCallable(
-    name = "short_path",
-    structField = true,
-    doc =
-        "The path of this file relative to its root. This excludes the aforementioned "
-            + "<i>root</i>, i.e. configuration-specific fragments of the path. This is also the "
-            + "path under which the file is mapped if it's in the runfiles of a binary."
-  )
   public final String getRootRelativePathString() {
     return getRootRelativePath().getPathString();
   }