Skip to content

Commit

Permalink
fix: Don't create console window when creating process
Browse files Browse the repository at this point in the history
  • Loading branch information
rzhao271 committed Jan 5, 2022
1 parent f1b0d30 commit 72b7027
Show file tree
Hide file tree
Showing 2 changed files with 30 additions and 0 deletions.
1 change: 1 addition & 0 deletions patches/node/.patches
Expand Up @@ -26,3 +26,4 @@ chore_fix_-wimplicit-fallthrough.patch
test_add_fixture_trim_option.patch
fix_crash_caused_by_gethostnamew_on_windows_7.patch
fix_suppress_clang_-wdeprecated-declarations_in_libuv.patch
fix_don_t_create_console_window_when_creating_process.patch
@@ -0,0 +1,29 @@
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
From: Raymond Zhao <raymondzhao@microsoft.com>
Date: Tue, 4 Jan 2022 16:11:41 -0800
Subject: fix: Don't create console window when creating process

This commit prevents console windows from being spawned during
execSync calls, or spawnSync calls where shell is true. Otherwise,
Windows users will see command prompts pop up for those calls.

The patch can be removed when the windowsHide parameter
for those calls is changed to have a default value of true.

This patch contains the same fix as
https://github.com/electron/electron/blob/13-x-y/patches/node/fix_don_t_create_console_window_when_creating_process.patch

diff --git a/deps/uv/src/win/process.c b/deps/uv/src/win/process.c
index 4038fbfd290d1acc91a27ad38e248a2deb06eb68..1ebc0f50e3f0cccb4d8887322b75e61c5284ba2f 100644
--- a/deps/uv/src/win/process.c
+++ b/deps/uv/src/win/process.c
@@ -1098,6 +1098,9 @@ int uv_spawn(uv_loop_t* loop,
process_flags |= DETACHED_PROCESS | CREATE_NEW_PROCESS_GROUP;
}

+ /* Don't create console window. */
+ process_flags |= CREATE_NO_WINDOW;
+
if (!CreateProcessW(application_path,
arguments,
NULL,

0 comments on commit 72b7027

Please sign in to comment.