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/process_wrap.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/process_wrap.cc')
| -rw-r--r-- | src/process_wrap.cc | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/src/process_wrap.cc b/src/process_wrap.cc index 7accc8c129e..6613a571891 100644 --- a/src/process_wrap.cc +++ b/src/process_wrap.cc @@ -211,6 +211,13 @@ class ProcessWrap : public HandleWrap { // options.stdio ParseStdioOptions(env, js_options, &options); + // options.windowsHide + Local<String> windows_hide_key = env->windows_hide_string(); + + if (js_options->Get(windows_hide_key)->IsTrue()) { + options.flags |= UV_PROCESS_WINDOWS_HIDE; + } + // options.windows_verbatim_arguments Local<String> windows_verbatim_arguments_key = env->windows_verbatim_arguments_string(); |