Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(32)

Unified Diff: utils/pub/io.dart

Issue 11091070: Change Process.start to return a future that completes with a (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: utils/pub/io.dart
diff --git a/utils/pub/io.dart b/utils/pub/io.dart
index bef2b2ae37e5f7d648c3203e3ad66e9c6cafec23..3a56edabb231e7ad4fa1de85b9b3fc8c1f7b4c1a 100644
--- a/utils/pub/io.dart
+++ b/utils/pub/io.dart
@@ -495,49 +495,48 @@ Future<PubProcessResult> runProcess(String executable, List<String> args,
}
options.environment = environment;
- final process = Process.start(executable, args, options);
-
- final outStream = new StringInputStream(process.stdout);
- final processStdout = <String>[];
-
- final errStream = new StringInputStream(process.stderr);
- final processStderr = <String>[];
-
final completer = new Completer<PubProcessResult>();
- checkComplete() {
- // Wait until the process is done and its output streams are closed.
- if (!pipeStdout && !outStream.closed) return;
- if (!pipeStderr && !errStream.closed) return;
- if (exitCode == null) return;
+ Process.start(executable, args, options).then((process) {
+ final outStream = new StringInputStream(process.stdout);
+ final processStdout = <String>[];
- completer.complete(new PubProcessResult(
- processStdout, processStderr, exitCode));
- }
+ final errStream = new StringInputStream(process.stderr);
+ final processStderr = <String>[];
- if (pipeStdout) {
- process.stdout.pipe(stdout, close: false);
- } else {
- outStream.onLine = () => processStdout.add(outStream.readLine());
- outStream.onClosed = checkComplete;
- outStream.onError = (error) => completer.completeException(error);
- }
+ checkComplete() {
+ // Wait until the process is done and its output streams are closed.
+ if (!pipeStdout && !outStream.closed) return;
+ if (!pipeStderr && !errStream.closed) return;
+ if (exitCode == null) return;
- if (pipeStderr) {
- process.stderr.pipe(stderr, close: false);
- } else {
- errStream.onLine = () => processStderr.add(errStream.readLine());
- errStream.onClosed = checkComplete;
- errStream.onError = (error) => completer.completeException(error);
- }
+ completer.complete(new PubProcessResult(
+ processStdout, processStderr, exitCode));
+ }
- process.onExit = (actualExitCode) {
- exitCode = actualExitCode;
- checkComplete();
- };
+ if (pipeStdout) {
+ process.stdout.pipe(stdout, close: false);
+ } else {
+ outStream.onLine = () => processStdout.add(outStream.readLine());
+ outStream.onClosed = checkComplete;
+ outStream.onError = (error) => completer.completeException(error);
+ }
+
+ if (pipeStderr) {
+ process.stderr.pipe(stderr, close: false);
+ } else {
+ errStream.onLine = () => processStderr.add(errStream.readLine());
+ errStream.onClosed = checkComplete;
+ errStream.onError = (error) => completer.completeException(error);
+ }
- process.onError = (error) => completer.completeException(error);
+ process.onExit = (actualExitCode) {
+ exitCode = actualExitCode;
+ checkComplete();
+ };
+ process.onError = (error) => completer.completeException(error);
+ });
return completer.future;
}
@@ -640,20 +639,16 @@ Future<bool> extractTarGz(InputStream stream, destination) {
return _extractTarGzWindows(stream, destination);
}
- var process = Process.start("tar",
- ["--extract", "--gunzip", "--directory", destination]);
var completer = new Completer<int>();
+ Process.start("tar",
+ ["--extract", "--gunzip", "--directory", destination]).then((process) {
- process.onExit = completer.complete;
- process.onError = completer.completeException;
-
- // Wait for the process to be fully started before writing to its
- // stdin stream.
- process.onStart = () {
+ process.onExit = completer.complete;
+ process.onError = completer.completeException;
stream.pipe(process.stdin);
process.stdout.pipe(stdout, close: false);
process.stderr.pipe(stderr, close: false);
- };
+ });
return completer.future.transform((exitCode) => exitCode == 0);
}

Powered by Google App Engine
This is Rietveld 408576698