Expose to Skylark Strings instead of PathFragments from CppConfiguration.

The only way to use those was in the `ctx.action(...executable...)`
parameter, made that accept string.

Fixes #2931.

RELNOTES: None.
PiperOrigin-RevId: 163511248

1 files changed, 4 insertions, 4 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/rules/SkylarkActionFactory.java b/src/main/java/com/google/devtools/build/lib/rules/SkylarkActionFactory.java
index 4360561628..2f87704d83 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/SkylarkActionFactory.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/SkylarkActionFactory.java
@@ -263,7 +263,7 @@ public class SkylarkActionFactory implements SkylarkValue {
               type = Object.class,
               allowedTypes = {
                   @ParamType(type = Artifact.class),
-                  @ParamType(type = PathFragment.class),
+                  @ParamType(type = String.class),
               },
               named = true,
               positional = false,
@@ -371,12 +371,12 @@ public class SkylarkActionFactory implements SkylarkValue {
       } else {
         builder.setExecutable(provider);
       }
-    } else if (executableUnchecked instanceof PathFragment) {
-      builder.setExecutable((PathFragment) executableUnchecked);
+    } else if (executableUnchecked instanceof String) {
+      builder.setExecutable(PathFragment.create((String) executableUnchecked));
     } else {
       throw new EvalException(
           null,
-          "expected file or PathFragment for "
+          "expected file or string for "
               + "executable but got "
               + EvalUtils.getDataTypeName(executableUnchecked)
               + " instead");