Skip to content
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

perf: patch libuv to use posix_spawn on macOS #27655

Conversation

trop[bot]
Copy link
Contributor

@trop trop bot commented Feb 8, 2021

Backport of #27026

See that PR for details.

Notes: Fixed slow child process spawning on macOS Big Sur

pdesantis and others added 4 commits February 8, 2021 15:25
patch libuv to fix a performance regression in macOS >= 11

Spawning child processes in an Electron application with a hardened
runtime has become slow in macOS Big Sur. This patch is a squashed
version of libuv/libuv#3064

This patch should be removed when libuv PR 3064 is merged.

Fixes: libuv/libuv#3050
Fixes: #26143
PR-URL: libuv/libuv#3064

Authored-by: Juan Pablo Canepa <jpcanepa@gmail.com>
Co-authored-by: Marcello Bastéa-Forte <marcello@descript.com>
Electron patch prepared by: Pat DeSantis <pdesantis3@gmail.com>
@trop trop bot requested a review from a team as a code owner February 8, 2021 15:25
@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Feb 8, 2021
@trop trop bot added 10-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes labels Feb 8, 2021
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened in the last 24 hours label Feb 8, 2021
@ckerr
Copy link
Member

ckerr commented Feb 10, 2021

elecctron-woa-testing failure appears to be unrelated to PR

@deepak1556 deepak1556 merged commit afb74be into 10-x-y Feb 11, 2021
@release-clerk
Copy link

release-clerk bot commented Feb 11, 2021

Release Notes Persisted

Fixed slow child process spawning on macOS Big Sur

@deepak1556 deepak1556 deleted the trop/10-x-y-bp-perf-patch-libuv-to-use-posix_spawn-on-macos-1612797938668 branch February 11, 2021 00:53
VerteDinde added a commit that referenced this pull request Feb 19, 2021
codebytere pushed a commit that referenced this pull request Feb 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
10-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants