From abfb016fccdfc8811d53fa9cbf404c0a222b56ec Mon Sep 17 00:00:00 2001
From: Andre Weinand <aweinand@microsoft.com>
Date: Fri, 25 Mar 2016 14:21:03 +0100
Subject: [PATCH] Revert "Support multi-line environment variables"

This reverts commit 4cd1c0fcebe91a06a215ff0d27b98ace720217da.
---
 src/vs/base/node/env.ts | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/vs/base/node/env.ts b/src/vs/base/node/env.ts
index b7c78d69803..1d801a661d9 100644
--- a/src/vs/base/node/env.ts
+++ b/src/vs/base/node/env.ts
@@ -19,8 +19,7 @@ export function getUserEnvironment(): TPromise<IEnv> {
 	}
 
 	return new TPromise((c, e) => {
-		// Use --null and split by '\0' as splitting by '\n' breaks multi-line environment variables
-		let child = cp.spawn(process.env.SHELL, ['-ilc', 'env', '--null'], {
+		let child = cp.spawn(process.env.SHELL, ['-ilc', 'env'], {
 			detached: true,
 			stdio: ['ignore', 'pipe', process.stderr],
 		});
@@ -38,7 +37,7 @@ export function getUserEnvironment(): TPromise<IEnv> {
 
 			let result: IEnv = Object.create(null);
 
-			buffer.split('\0').forEach(line => {
+			buffer.split('\n').forEach(line => {
 				let pos = line.indexOf('=');
 				if (pos > 0) {
 					let key = line.substring(0, pos);
-- 
GitLab