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

Unified Diff: sdk/lib/_internal/pub/lib/src/io.dart

Issue 17261026: Include parent environment by default, add option to not, for Process. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 6 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: sdk/lib/_internal/pub/lib/src/io.dart
diff --git a/sdk/lib/_internal/pub/lib/src/io.dart b/sdk/lib/_internal/pub/lib/src/io.dart
index 8b6c7fdd760595beba2ed3da4515ef87281aa391..2d1a8bb928e25d93a862d4522292a4d2c7994df6 100644
--- a/sdk/lib/_internal/pub/lib/src/io.dart
+++ b/sdk/lib/_internal/pub/lib/src/io.dart
@@ -591,19 +591,12 @@ Future _doProcess(Function fn, String executable, List<String> args,
executable = "cmd";
}
- var env = null;
- if (environment != null) {
- env = new Map.from(Platform.environment);
- environment.forEach((key, value) => env[key] = value);
- }
-
-
log.process(executable, args);
return fn(executable,
args,
workingDirectory: workingDir,
- environment: env);
+ environment: environment);
}
/// Wraps [input] to provide a timeout. If [input] completes before

Powered by Google App Engine
This is Rietveld 408576698