mirror of
https://github.com/nodejs/node.git
synced 2026-11-09 15:11:11 -05:00
child_process: kHideConsoleWindows for spawnSync
kHideConsoleWindows should hide console windows for execSync and spawnSync calls as well. This fix adds on to https://github.com/nodejs/node/pull/39712. PR-URL: https://github.com/nodejs/node/pull/41412 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
This commit is contained in:
1 parent
998e5acc69
commit
863d13c192
1 file changed
+3
@@ -810,6 +810,9 @@ Maybe<int> SyncProcessRunner::ParseOptions(Local<Value> js_value) {
|
||||
if (js_win_hide->BooleanValue(isolate))
|
||||
uv_process_options_.flags |= UV_PROCESS_WINDOWS_HIDE;
|
||||
|
||||
if (env()->hide_console_windows())
|
||||
uv_process_options_.flags |= UV_PROCESS_WINDOWS_HIDE_CONSOLE;
|
||||
|
||||
Local<Value> js_wva =
|
||||
js_options->Get(context, env()->windows_verbatim_arguments_string())
|
||||
.ToLocalChecked();
|
||||
|
||||
Reference in new issue
Block a user