child_process: execFileSync stderr should inherit

If you don't set your options.stdio for execSync and execFileSync
capture and write to stderr of the parent process by default.

Fixes #7110

1 files changed, 12 insertions, 1 deletions

diff --git a/lib/child_process.js b/lib/child_process.js
index 1f121d901f..f606224856 100644
--- a/lib/child_process.js
+++ b/lib/child_process.js
@@ -1304,7 +1304,13 @@ function checkExecSyncError(ret) {
 
 
 function execFileSync(/*command, options*/) {
-  var ret = spawnSync.apply(null, arguments);
+  var opts = normalizeSpawnArguments.apply(null, arguments);
+  var inheritStderr = !!!opts.options.stdio;
+
+  var ret = spawnSync(opts.file, opts.args.slice(1), opts.options);
+
+  if (inheritStderr)
+    process.stderr.write(ret.stderr);
 
   var err = checkExecSyncError(ret);
 
@@ -1318,9 +1324,14 @@ exports.execFileSync = execFileSync;
 
 function execSync(/*comand, options*/) {
   var opts = normalizeExecArgs.apply(null, arguments);
+  var inheritStderr = opts.options ? !!!opts.options.stdio : true;
+
   var ret = spawnSync(opts.file, opts.args, opts.options);
   ret.cmd = opts.cmd;
 
+  if (inheritStderr)
+    process.stderr.write(ret.stderr);
+
   var err = checkExecSyncError(ret);
 
   if (err)