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 #27654

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 11-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

electron-woa-testing failure appears to be a flake unrelated to PR

@deepak1556 deepak1556 merged commit 19f8ea3 into 11-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/11-x-y-bp-perf-patch-libuv-to-use-posix_spawn-on-macos-1612797939870 branch February 11, 2021 00:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
11-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