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.

Sign up for GitHub

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

Jump to bottom

deps: update V8 to 12.9 #54536

Closed
wants to merge 13 commits into from
Closed

deps: update V8 to 12.9 #54536

wants to merge 13 commits into from

Conversation

targos
Copy link
Member

@targos targos commented Aug 24, 2024 •
edited
Loading

Notable changes:

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/gyp
  • @nodejs/security-wg
  • @nodejs/v8-update

@nodejs-github-bot nodejs-github-bot added build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. tools Issues and PRs related to the tools directory. v8 engine Issues and PRs related to the V8 dependency. labels Aug 24, 2024
@targos targos added semver-major PRs that contain breaking changes and should be released in the next major version. request-ci Add this label to start a Jenkins CI on a PR. labels Aug 24, 2024
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Aug 24, 2024
@nodejs-github-bot

This comment was marked as outdated.

Sign in to view
@nodejs-github-bot

This comment was marked as outdated.

Sign in to view
@nodejs-github-bot

This comment was marked as outdated.

Sign in to view
@nodejs-github-bot

This comment was marked as outdated.

Sign in to view
@targos
Copy link
Member Author

targos commented Aug 24, 2024

Updated d73e710 to fix the MSVC error.
Opened https://chromium-review.googlesource.com/c/v8/v8/+/5805956 upstream to fix the macOS Clang error.

@targos
Copy link
Member Author

targos commented Aug 24, 2024

@nodejs/platform-aix Can you have a look at https://ci.nodejs.org/job/node-test-commit-aix/53241/nodes=aix72-ppc64/console ?

@avivkeller avivkeller added the dependencies Pull requests that update a dependency file. label Aug 24, 2024
@targos targos added the request-ci Add this label to start a Jenkins CI on a PR. label Aug 26, 2024
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Aug 26, 2024
@nodejs-github-bot
Copy link
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/61508/

@nodejs-github-bot
Copy link
Collaborator

V8 CI: https://ci.nodejs.org/job/node-test-commit-v8-linux/nodes=benchmark-ubuntu2204-intel-64,v8test=v8test/6156/

@nodejs-github-bot
Copy link
Collaborator

V8 CI: https://ci.nodejs.org/job/node-test-commit-v8-linux/nodes=rhel8-s390x,v8test=v8test/6156/

@nodejs-github-bot
Copy link
Collaborator

V8 CI: https://ci.nodejs.org/job/node-test-commit-v8-linux/nodes=rhel8-ppc64le,v8test=v8test/6156/

@targos
Copy link
Member Author

targos commented Aug 27, 2024

https://ci.nodejs.org/job/node-test-commit-osx/60761/

@targos
Copy link
Member Author

targos commented Aug 27, 2024 •
edited
Loading

https://ci.nodejs.org/job/node-test-commit-windows-fanned/65162/
https://ci.nodejs.org/job/node-compile-windows-debug/23534/

@targos
Copy link
Member Author

targos commented Aug 27, 2024

https://ci.nodejs.org/job/node-test-commit-osx/60768/

@abmusse
Copy link
Contributor

abmusse commented Aug 27, 2024

@nodejs/platform-aix Can you have a look at https://ci.nodejs.org/job/node-test-commit-aix/53241/nodes=aix72-ppc64/console ?

Hello @targos I've opened a CL on v8 to resolve that build issue https://chromium-review.googlesource.com/c/v8/v8/+/5789180

@targos
Copy link
Member Author

targos commented Aug 27, 2024 •
edited
Loading

@nodejs/platform-windows, somehow Windows debug builds are broken. I tried two runs to be sure, but they both ended up in a timeout while building an addon: https://ci.nodejs.org/job/node-compile-windows-debug/23534/nodes=win-vs2022/console

@StefanStojanovic
Copy link
Contributor

@nodejs/platform-windows, somehow Windows debug builds are broken. I tried to runs to be sure, but they both ended up in a timeout while building an addon: https://ci.nodejs.org/job/node-compile-windows-debug/23534/nodes=win-vs2022/console

I'll look into it.

@StefanStojanovic
Copy link
Contributor

So what happens when you run that command (for me it is .\Debug\node.exe deps\npm\node_modules\node-gyp\bin\node-gyp rebuild --directory=test\addons\hello-world --nodedir="E:\work\node") after compiling in debug mode, is that you get this:

image

Clicking Ignore will get you through it, successfully but in CI we cannot do it. I'm not 100% sure, but I think we had this issue with some of the v12.x updates, Will try to find it.

nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this pull request Dec 15, 2024
@StefanStojanovic @targos @nodejs-github-bot
deps: patch V8 to support compilation with MSVC
87bcf3f 
Co-Authored-By: Michaël Zasso <[email protected]>
PR-URL: nodejs/node#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this pull request Dec 15, 2024
@targos @nodejs-github-bot
deps: patch V8 to support older Clang versions
0a67f48 
PR-URL: nodejs/node#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this pull request Dec 16, 2024
@StefanStojanovic @targos @nodejs-github-bot
deps: patch V8 to support compilation with MSVC
61a5828 
Co-Authored-By: Michaël Zasso <[email protected]>
PR-URL: nodejs/node#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this pull request Dec 16, 2024
@targos @nodejs-github-bot
deps: patch V8 to support older Clang versions
c442c8e 
PR-URL: nodejs/node#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this pull request Dec 17, 2024
@StefanStojanovic @targos @nodejs-github-bot
deps: patch V8 to support compilation with MSVC
5fe32a7 
Co-Authored-By: Michaël Zasso <[email protected]>
PR-URL: nodejs/node#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this pull request Dec 17, 2024
@targos @nodejs-github-bot
deps: patch V8 to support older Clang versions
1d7af09 
PR-URL: nodejs/node#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this pull request Dec 18, 2024
@StefanStojanovic @targos @nodejs-github-bot
deps: patch V8 to support compilation with MSVC
96f4970 
Co-Authored-By: Michaël Zasso <[email protected]>
PR-URL: nodejs/node#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this pull request Dec 18, 2024
@targos @nodejs-github-bot
deps: patch V8 to support older Clang versions
b848d0c 
PR-URL: nodejs/node#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
targos added a commit that referenced this pull request Dec 19, 2024
@StefanStojanovic @targos
deps: patch V8 to support compilation with MSVC
47c5c31 
Co-Authored-By: Michaël Zasso <[email protected]>
PR-URL: #54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
targos added a commit that referenced this pull request Dec 19, 2024
@targos
deps: patch V8 to support older Clang versions
931e793 
PR-URL: #54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this pull request Dec 19, 2024
@StefanStojanovic @targos @nodejs-github-bot
deps: patch V8 to support compilation with MSVC
3ab7d3a 
Co-Authored-By: Michaël Zasso <[email protected]>
PR-URL: nodejs/node#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this pull request Dec 19, 2024
@targos @nodejs-github-bot
deps: patch V8 to support older Clang versions
88c5023 
PR-URL: nodejs/node#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
targos added a commit to targos/node that referenced this pull request Dec 19, 2024
@StefanStojanovic @targos
deps: patch V8 to support compilation with MSVC
31ea00c 
Co-Authored-By: Michaël Zasso <[email protected]>
PR-URL: nodejs#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
targos added a commit to targos/node that referenced this pull request Dec 19, 2024
@targos
deps: patch V8 to support older Clang versions
1fa7c32 
PR-URL: nodejs#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
targos added a commit that referenced this pull request Dec 20, 2024
@StefanStojanovic @targos
deps: patch V8 to support compilation with MSVC
149300b 
Co-Authored-By: Michaël Zasso <[email protected]>
PR-URL: #54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
targos added a commit that referenced this pull request Dec 20, 2024
@targos
deps: patch V8 to support older Clang versions
024b846 
PR-URL: #54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this pull request Dec 20, 2024
@StefanStojanovic @targos @nodejs-github-bot
deps: patch V8 to support compilation with MSVC
b13cddf 
Co-Authored-By: Michaël Zasso <[email protected]>
PR-URL: nodejs/node#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this pull request Dec 20, 2024
@targos @nodejs-github-bot
deps: patch V8 to support older Clang versions
dbaf9be 
PR-URL: nodejs/node#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
targos added a commit to targos/node that referenced this pull request Dec 20, 2024
@StefanStojanovic @targos
deps: patch V8 to support compilation with MSVC
7fccfb3 
Co-Authored-By: Michaël Zasso <[email protected]>
PR-URL: nodejs#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
targos added a commit to targos/node that referenced this pull request Dec 20, 2024
@targos
deps: patch V8 to support older Clang versions
13f6c7b 
PR-URL: nodejs#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
targos added a commit that referenced this pull request Dec 21, 2024
@StefanStojanovic @targos
deps: patch V8 to support compilation with MSVC
4a9dbfb 
Co-Authored-By: Michaël Zasso <[email protected]>
PR-URL: #54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
targos added a commit that referenced this pull request Dec 21, 2024
@targos
deps: patch V8 to support older Clang versions
11d14f8 
PR-URL: #54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this pull request Dec 21, 2024
@StefanStojanovic @targos @nodejs-github-bot
deps: patch V8 to support compilation with MSVC
e99e8c9 
Co-Authored-By: Michaël Zasso <[email protected]>
PR-URL: nodejs/node#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this pull request Dec 21, 2024
@targos @nodejs-github-bot
deps: patch V8 to support older Clang versions
377f936 
PR-URL: nodejs/node#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this pull request Dec 22, 2024
@StefanStojanovic @targos @nodejs-github-bot
deps: patch V8 to support compilation with MSVC
4e93cd6 
Co-Authored-By: Michaël Zasso <[email protected]>
PR-URL: nodejs/node#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this pull request Dec 22, 2024
@targos @nodejs-github-bot
deps: patch V8 to support older Clang versions
912a803 
PR-URL: nodejs/node#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this pull request Dec 23, 2024
@StefanStojanovic @targos @nodejs-github-bot
deps: patch V8 to support compilation with MSVC
e35da3d 
Co-Authored-By: Michaël Zasso <[email protected]>
PR-URL: nodejs/node#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this pull request Dec 23, 2024
@targos @nodejs-github-bot
deps: patch V8 to support older Clang versions
f8b5e86 
PR-URL: nodejs/node#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this pull request Dec 24, 2024
@StefanStojanovic @targos @nodejs-github-bot
deps: patch V8 to support compilation with MSVC
14c632d 
Co-Authored-By: Michaël Zasso <[email protected]>
PR-URL: nodejs/node#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this pull request Dec 24, 2024
@targos @nodejs-github-bot
deps: patch V8 to support older Clang versions
97e0506 
PR-URL: nodejs/node#54536
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@marco-ippolito marco-ippolito marco-ippolito approved these changes

@aduh95 aduh95 aduh95 approved these changes

@richardlau richardlau richardlau approved these changes

@gengjiawen gengjiawen gengjiawen approved these changes

Assignees
No one assigned
Labels
build Issues and PRs related to build files or the CI. commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits. dependencies Pull requests that update a dependency file. needs-ci PRs that need a full CI run. semver-major PRs that contain breaking changes and should be released in the next major version. tools Issues and PRs related to the tools directory. v8 engine Issues and PRs related to the V8 dependency.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@targos @nodejs-github-bot @abmusse @StefanStojanovic @richardlau @gengjiawen @aduh95 @marco-ippolito @avivkeller