diff options
author | cjihrig <cjihrig@gmail.com> | 2017-09-13 07:22:45 +0300 |
---|---|---|
committer | cjihrig <cjihrig@gmail.com> | 2017-10-16 23:19:54 +0300 |
commit | 3b7a9a2589a55f3c4cb9abc2162351b22aec6973 (patch) | |
tree | 23c84eb5822bc1712b139ca71f692f8047a4b8f7 /src/spawn_sync.cc | |
parent | e2015b53477845513d87a6750a87019b37213943 (diff) |
child_process: add windowsHide option
This commit exposes the UV_PROCESS_WINDOWS_HIDE flag in Node
as a windowsHide option to the child_process methods. The
option is only applicable to Windows, and is ignored elsewhere.
Fixes: https://github.com/nodejs/node/issues/15217
PR-URL: https://github.com/nodejs/node/pull/15380
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
Diffstat (limited to 'src/spawn_sync.cc')
-rw-r--r-- | src/spawn_sync.cc | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/src/spawn_sync.cc b/src/spawn_sync.cc index 430bbe3b8a3..c9765f51b26 100644 --- a/src/spawn_sync.cc +++ b/src/spawn_sync.cc @@ -777,6 +777,11 @@ int SyncProcessRunner::ParseOptions(Local<Value> js_value) { if (js_options->Get(env()->detached_string())->BooleanValue()) uv_process_options_.flags |= UV_PROCESS_DETACHED; + Local<String> win_hide = env()->windows_hide_string(); + + if (js_options->Get(win_hide)->BooleanValue()) + uv_process_options_.flags |= UV_PROCESS_WINDOWS_HIDE; + Local<String> wba = env()->windows_verbatim_arguments_string(); if (js_options->Get(wba)->BooleanValue()) |