Add a wrapper for RunWithManagedStdio

Bug: b/434043969

Author: Databean

base/cvd/cuttlefish/common/libs/utils/BUILD.bazel

@@ -349,6 +349,7 @@ cf_cc_library(
     hdrs = ["subprocess_managed_stdio.h"],
     deps = [
         "//cuttlefish/common/libs/fs",
+        "//cuttlefish/common/libs/utils:result",
         "//cuttlefish/common/libs/utils:subprocess",
         "//libbase",
     ],

base/cvd/cuttlefish/common/libs/utils/subprocess_managed_stdio.cc

@@ -51,7 +51,7 @@ class ThreadJoiner {
 
 }  // namespace
 
-int RunWithManagedStdio(Command&& cmd_tmp, const std::string* stdin_str,
+int RunWithManagedStdio(Command cmd_tmp, const std::string* stdin_str,
                         std::string* stdout_str, std::string* stderr_str,
                         SubprocessOptions options) {
   /*
@@ -140,4 +140,18 @@ int RunWithManagedStdio(Command&& cmd_tmp, const std::string* stdin_str,
   return code;
 }
 
+Result<std::string> RunAndCaptureStdout(Command command) {
+  std::string standard_out;
+  std::string standard_err;
+  std::string command_str = command.GetShortName();
+  int exit_code = RunWithManagedStdio(std::move(command), nullptr,
+                                      &standard_out, &standard_err);
+  LOG(DEBUG) << "Ran " << command_str << " with stdout:\n" << standard_out;
+  LOG(DEBUG) << "Ran " << command_str << " with stderr:\n" << standard_err;
+  CF_EXPECTF(exit_code == 0,
+             "Failed to execute '{}' <args>: exit code = {}, stdout = '{}', stderr = '{}'",
+             command_str, exit_code, standard_out, standard_err);
+  return standard_out;
+}
+
 }  // namespace cuttlefish

base/cvd/cuttlefish/common/libs/utils/subprocess_managed_stdio.h

@@ -17,10 +17,14 @@
 
 #include <string>
 
+#include "cuttlefish/common/libs/utils/result.h"
 #include "cuttlefish/common/libs/utils/subprocess.h"
 
 namespace cuttlefish {
 
+// Success is defined by the exit code being `0`.
+Result<std::string> RunAndCaptureStdout(Command);
+
 /*
  * Consumes a Command and runs it, optionally managing the stdio channels.
  *
@@ -33,8 +37,8 @@ namespace cuttlefish {
  * If some setup fails, `command` fails to start, or `command` exits due to a
  * signal, the return value will be negative.
  */
-int RunWithManagedStdio(Command&& cmd_tmp, const std::string* stdin,
-                        std::string* stdout, std::string* stderr,
+int RunWithManagedStdio(Command, const std::string* stdin, std::string* stdout,
+                        std::string* stderr,
                         SubprocessOptions options = SubprocessOptions());
 
 }  // namespace cuttlefish