Skip to content

build: remove deps_add_v8_object_setinternalfieldfornodecore.patch#46992

Merged
codebytere merged 1 commit intomainfrom
chore/remove-deps_add_v8_object_setinternalfieldfornodecore.patch
May 8, 2025
Merged

build: remove deps_add_v8_object_setinternalfieldfornodecore.patch#46992
codebytere merged 1 commit intomainfrom
chore/remove-deps_add_v8_object_setinternalfieldfornodecore.patch

Conversation

@ckerr
Copy link
Copy Markdown
Member

@ckerr ckerr commented May 8, 2025

Description of Change

This was a Node 20-specific workaround. I think it's no longer needed in Node v22?

Xref: nodejs/node#49874.

Checklist

Release Notes

Notes: none.

This was a Node 20-specific workaround that's no longer needed in Node 22.

Xref: nodejs/node#49874.
@ckerr ckerr requested a review from codebytere May 8, 2025 04:25
@ckerr ckerr requested a review from a team as a code owner May 8, 2025 04:25
@ckerr ckerr added component/node-integration semver/none target/36-x-y PR should also be added to the "36-x-y" branch. target/37-x-y PR should also be added to the "37-x-y" branch. labels May 8, 2025
@electron-cation electron-cation Bot added the new-pr 🌱 PR opened recently label May 8, 2025
@codebytere codebytere changed the title chore: remove deps_add_v8_object_setinternalfieldfornodecore.patch build: remove deps_add_v8_object_setinternalfieldfornodecore.patch May 8, 2025
@codebytere codebytere merged commit 24f1f7e into main May 8, 2025
68 checks passed
@codebytere codebytere deleted the chore/remove-deps_add_v8_object_setinternalfieldfornodecore.patch branch May 8, 2025 07:40
@release-clerk
Copy link
Copy Markdown

release-clerk Bot commented May 8, 2025

No Release Notes

@trop
Copy link
Copy Markdown
Contributor

trop Bot commented May 8, 2025

I was unable to backport this PR to "36-x-y" cleanly;
you will need to perform this backport manually.

@trop trop Bot added needs-manual-bp/36-x-y and removed target/36-x-y PR should also be added to the "36-x-y" branch. labels May 8, 2025
@trop
Copy link
Copy Markdown
Contributor

trop Bot commented May 8, 2025

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

@trop trop Bot added in-flight/37-x-y merged/37-x-y PR was merged to the "37-x-y" branch. and removed target/37-x-y PR should also be added to the "37-x-y" branch. in-flight/37-x-y labels May 8, 2025
@trop
Copy link
Copy Markdown
Contributor

trop Bot commented May 21, 2025

@ckerr has manually backported this PR to "36-x-y", please check out #47195

@trop trop Bot added in-flight/36-x-y merged/36-x-y PR was merged to the "36-x-y" branch. and removed needs-manual-bp/36-x-y in-flight/36-x-y labels May 21, 2025
kigh-ota pushed a commit to kigh-ota/electron that referenced this pull request Sep 30, 2025
…lectron#46992)

chore: remove deps_add_v8_object_setinternalfieldfornodecore.patch

This was a Node 20-specific workaround that's no longer needed in Node 22.

Xref: nodejs/node#49874.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/node-integration merged/36-x-y PR was merged to the "36-x-y" branch. merged/37-x-y PR was merged to the "37-x-y" branch. new-pr 🌱 PR opened recently semver/none

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants