Skip to content

[v16.x] deps: revert ABI-breaking change from V8 9.2 - #39624

Closed
targos wants to merge 1 commit into
nodejs:v16.x-stagingfrom
targos:fix-39623
Closed

[v16.x] deps: revert ABI-breaking change from V8 9.2#39624
targos wants to merge 1 commit into
nodejs:v16.x-stagingfrom
targos:fix-39623

Conversation

@targos

Copy link
Copy Markdown
Member

@nodejs-github-botnodejs-github-bot added build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. v16.x v8 engine Issues and PRs related to the V8 dependency. labels Aug 2, 2021
@targos

Copy link
Copy Markdown
MemberAuthor

/cc @addaleax@reuben

@targostargos added the fast-track PRs that do not need to wait for 48 hours to land. label Aug 2, 2021
@github-actions

Copy link
Copy Markdown
Contributor

Fast-track has been requested by @targos. Please 👍 to approve.

@targos

Copy link
Copy Markdown
MemberAuthor

I'd like to fast-track this and include it in a small patch release tomorrow.

@targostargos added the request-ci Add this label to start a Jenkins CI on a PR. label Aug 2, 2021
@targos

Copy link
Copy Markdown
MemberAuthor

@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Aug 2, 2021
@nodejs-github-bot

nodejs-github-bot commented Aug 2, 2021

Copy link
Copy Markdown
Collaborator

@targostargos changed the title deps: revert ABI-breaking change from V8 9.2[v16.x] deps: revert ABI-breaking change from V8 9.2Aug 2, 2021
@nodejs-github-bot

nodejs-github-bot commented Aug 2, 2021

Copy link
Copy Markdown
Collaborator

@targos

Copy link
Copy Markdown
MemberAuthor

Landed in 41e105b29438499ec024820b4bbfc6797cbf9311

@targostargos closed this Aug 2, 2021
@targos
targos deleted the fix-39623 branch August 2, 2021 17:31
targos added a commit that referenced this pull request Aug 2, 2021
Refs: v8/v8@a7980d4
Refs: v8/v8@ad4eab0Fixes: #39623
PR-URL: #39624
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Beth Griggs <bgriggs@redhat.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
targos added a commit that referenced this pull request Aug 2, 2021
Notable changes:
* Updated npm to 7.20.3 (npm team)
#39579
* Reverted an ABI-breaking change from V8 9.2 that could impact some
native modules (Michaël Zasso)
#39624
* Fixed a bug in error handling known to affect at least Webpack and
Jest (Guy Bedford) #39593
PR-URL: #39631
@targostargos mentioned this pull request Aug 2, 2021
targos added a commit that referenced this pull request Aug 3, 2021
Notable changes:
* Updated npm to 7.20.3 (npm team)
#39579
* Reverted an ABI-breaking change from V8 9.2 that could impact some
native modules (Michaël Zasso)
#39624
* Fixed a bug in error handling known to affect at least Webpack and
Jest (Guy Bedford) #39593
PR-URL: #39631
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

buildIssues and PRs related to build files or the CI.fast-trackPRs that do not need to wait for 48 hours to land.needs-ciPRs that need a full CI run.v8 engineIssues and PRs related to the V8 dependency.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@targos@nodejs-github-bot@addaleax@richardlau@gireeshpunathil@BethGriggs@aduh95@Flarna