Automated rollback of commit abe1f66.

*** Reason for rollback ***

b/369785485#comment12

*** Original change description ***

Extract a `HasExecPath` interface from `ActionInput`.

PiperOrigin-RevId: 680556719
Change-Id: I8410c9b4b9aeb462f51dd1fdbcb02d08f8cc5d1a

src/main/java/com/google/devtools/build/lib/actions/ActionInput.java

@@ -19,16 +19,26 @@
 /**
  * Represents an input file to a build action, with an appropriate relative path.
  *
- * <p>Artifact is the only notable implementer of the interface, but the interface remains because
- * 1) some Google specific rules ship files that could be Artifacts to remote execution by
- * instantiating ad-hoc derived classes of ActionInput. 2) historically, Google C++ rules allow
- * underspecified C++ builds. For that case, we have extra logic to guess the undeclared header
- * inclusions (eg. computed inclusions). The extra logic lives in a file that is not needed for
- * remote execution, but is a dependency, and it is inserted as a non-Artifact ActionInput.
+ * <p>Artifact is the only notable implementer of the interface, but the interface remains
+ * because 1) some Google specific rules ship files that could be Artifacts to remote execution
+ * by instantiating ad-hoc derived classes of ActionInput.  2) historically, Google C++ rules
+ * allow underspecified C++ builds. For that case, we have extra logic to guess the undeclared
+ * header inclusions (eg. computed inclusions). The extra logic lives in a file that is not
+ * needed for remote execution, but is a dependency, and it is inserted as a non-Artifact
+ * ActionInput.
+ *
+ * <p>ActionInput is used as a cache "key" for ActionInputFileCache: for Artifacts, the
+ * digest/size is already stored in Artifact, but for non-artifacts, we use getExecPathString
+ * to find this data in a filesystem related cache.
  */
-public interface ActionInput extends HasExecPathString {
+public interface ActionInput {
+
+  /** @return the relative path to the input file. */
+  String getExecPathString();
 
-  /** Same as {@link #getExecPathString}, but returns a {@link PathFragment}. */
+  /**
+   * @return the relative path to the input file.
+   */
   PathFragment getExecPath();
 
   /** The input is a directory. */

src/main/java/com/google/devtools/build/lib/actions/BUILD

@@ -317,7 +317,6 @@ java_library(
         "ArtifactResolver.java",
         "ArtifactRoot.java",
         "Artifacts.java",
-        "HasExecPathString.java",
         "PathMapper.java",
     ],
     deps = [