Add Process.shell and Process.runShell.

sdk/lib/io/process.dart

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 part of dart.io;
 
 // TODO(ager): The only reason for this class is that we
 // cannot patch a top-level at this point.
 class _ProcessUtils {
   external static void _exit(int status);
@@ skipping 82 matching lines @@
    *
    * Returns a [:Future<ProcessResult>:] that completes with the
    * result of running the process, i.e., exit code, standard out and
    * standard in.
    */
   external static Future<ProcessResult> run(String executable,
                                             List<String> arguments,
                                             [ProcessOptions options]);
 
   /**
+   * Starts a process in the system shell and runs it non-interactively to
+   * completion.
+   *
+   * On Linux and Mac OS, [:/bin/sh:] is used to execute the [executable].
+   * On Windows, [:%WINDIR%\system32\cmd.exe:] is used.
+   *
+   * An optional [ProcessOptions] object can be passed to specify
+   * options other than the executable and the arguments.
+   *
+   * Returns a [:Future<ProcessResult>:] that completes with the
+   * result of running the process, i.e., exit code, standard out and
+   * standard in.
+   */
+  static Future<ProcessResult> runShell(String executable,
+                                        List<String> arguments,
+                                        [ProcessOptions options])
+      => run(_getShellCommand(),
+             _getShellArguments(executable, arguments),
+             options);
+
+  static String _getShellCommand() {
+    if (Platform.operatingSystem == 'windows') {
+      return 'cmd.exe';
+    }
+    return '/bin/sh';
+  }
+
+  static List<String> _getShellArguments(String executable,
+                                         List<String> arguments) {
+    List<String> shellArguments = [];
+    if (Platform.operatingSystem == 'windows') {
+      shellArguments.add('/c');
+      shellArguments.add(executable);
+      for (var arg in arguments) {
+        arg = arg.replaceAll('"', r'\"');
+        shellArguments.add(arg);
+      }
+    } else {
+      var commandLine = new StringBuffer();
+      commandLine.write(executable);
+      shellArguments.add("-c");
+      for (var arg in arguments) {
+        arg = arg.replaceAll("'", "'\"'\"'");

[kustermann, 2013/05/22 11:54:26]

I think there is something fishy with this replacement.
  arg'  --converts-to--> arg'"'"'
  arg\' --converts-to--> arg\'"'"'
Is this what we want?

(I'm also unsure how cmd.exe treats such strings.)

[Anders Johnsen, 2013/05/22 12:58:25]

See https://codereview.chromium.org/15743002

+        commandLine.write(" '$arg'");
+      }
+      shellArguments.add(commandLine.toString());
+    }
+    return shellArguments;
+  }
+
+  /**
    * Returns the standard output stream of the process as a [:Stream:].
    *
    * Throws an [UnsupportedError] if the process is
    * non-interactive.
    */
   Stream<List<int>> get stdout;
 
   /**
    * Returns the standard error stream of the process as a [:Stream:].
    *
@@ skipping 177 matching lines @@
   /**
    * Contains the system message for the process exception if any.
    */
   final String message;
 
   /**
    * Contains the OS error code for the process exception if any.
    */
   final int errorCode;
 }