New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Don't create console window when creating process #32340
Conversation
bed7a42
to
72b7027
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This patch was removed to adopt the embedder flag provided in nodejs/node#39712
It looks like the spawn_sync path was missed in the upstream PR. This should be addressed there.
/cc @zcbenz
a61327d
to
a94acd3
Compare
a94acd3
to
52a6b9e
Compare
patches/node/fix_don_t_create_console_window_when_creating_process.patch
Show resolved
Hide resolved
The upstream node patch is being tracked at nodejs/node#41412 and seems nearly ready to merge. |
patches/node/fix_don_t_create_console_window_when_creating_process.patch
Outdated
Show resolved
Hide resolved
…cess.patch Co-authored-by: Robo <hop2deep@gmail.com>
Release Notes Persisted
|
I have automatically backported this PR to "17-x-y", please check out #32378 |
I have automatically backported this PR to "16-x-y", please check out #32379 |
* fix: Don't create console window when creating process * Update patches/node/fix_don_t_create_console_window_when_creating_process.patch Co-authored-by: Robo <hop2deep@gmail.com> * Remove extra line in description Co-authored-by: Robo <hop2deep@gmail.com>
Description of Change
Since the
windowsHide
parameters still default tofalse
, we need to keep the patch that disables console windows from showing up. Otherwise, forexecSync
calls, orspawnSync
calls withshell: true
, console windows still show up for Windows users.This issue has been occurring since Electron 16.
Downstream issue: microsoft/vscode#138792
CC @deepak1556
Checklist
npm test
passesRelease Notes
Notes: Fix regression where console windows would open for execSync and some spawnSync calls for Windows users.