Skip to content

fix: libuv patches to address child_process.spawn slowness#33337

Merged
deepak1556 merged 2 commits intomainfrom
patch-libuv
Mar 23, 2022
Merged

fix: libuv patches to address child_process.spawn slowness#33337
deepak1556 merged 2 commits intomainfrom
patch-libuv

Conversation

@nornagon
Copy link
Copy Markdown
Contributor

Description of Change

Closes #26143.

This backports six PRs from libuv 1.44.1 to fix slow child_process spawning
behavior on macOS Big Sur and later.

libuv references:

Checklist

Release Notes

Notes: Fixed slowness when using child_process.spawn and related methods on macOS Big Sur and newer.

@nornagon nornagon requested review from a team as code owners March 18, 2022 18:40
@nornagon nornagon added semver/patch backwards-compatible bug fixes target/15-x-y labels Mar 18, 2022
@electron-cation electron-cation bot added new-pr 🌱 PR opened recently and removed new-pr 🌱 PR opened recently labels Mar 18, 2022
@deepak1556
Copy link
Copy Markdown
Member

deepak1556 commented Mar 21, 2022

Copy link
Copy Markdown
Member

@deepak1556 deepak1556 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Marking to avoid accidental merge.

@deepak1556
Copy link
Copy Markdown
Member

Failing tests are unrelated, merging.

@deepak1556 deepak1556 merged commit f912130 into main Mar 23, 2022
@deepak1556 deepak1556 deleted the patch-libuv branch March 23, 2022 13:30
@release-clerk
Copy link
Copy Markdown

release-clerk bot commented Mar 23, 2022

Release Notes Persisted

Fixed slowness when using child_process.spawn and related methods on macOS Big Sur and newer.

@trop
Copy link
Copy Markdown
Contributor

trop bot commented Mar 23, 2022

I have automatically backported this PR to "16-x-y", please check out #33405

@trop
Copy link
Copy Markdown
Contributor

trop bot commented Mar 23, 2022

I have automatically backported this PR to "15-x-y", please check out #33406

@trop trop bot removed the target/16-x-y label Mar 23, 2022
@trop
Copy link
Copy Markdown
Contributor

trop bot commented Mar 23, 2022

I have automatically backported this PR to "18-x-y", please check out #33407

@trop
Copy link
Copy Markdown
Contributor

trop bot commented Mar 23, 2022

I have automatically backported this PR to "17-x-y", please check out #33408

bavulapati pushed a commit to bavulapati/electron that referenced this pull request Apr 29, 2022
…33337)

* fix: libuv patches to address child_process.spawn slowness

* chore: backport additional patches

Co-authored-by: deepak1556 <[email protected]>
khalwa pushed a commit to solarwindscloud/electron that referenced this pull request Feb 22, 2023
…33337)

* fix: libuv patches to address child_process.spawn slowness

* chore: backport additional patches

Co-authored-by: deepak1556 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver/patch backwards-compatible bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

child_process spawn locks up renderer for 300 - 3000ms on signed app on macOS Big Sur

3 participants