Skip to content

tools: filter V8 scripts for build toolchain#63069

Open
richardlau wants to merge 1 commit intonodejs:mainfrom
richardlau:makev8
Open

tools: filter V8 scripts for build toolchain#63069
richardlau wants to merge 1 commit intonodejs:mainfrom
richardlau:makev8

Conversation

@richardlau
Copy link
Copy Markdown
Member

Patch the build scripts for V8 when running the V8 CI for Linux on ppc64le and s390x to workaround inconsistencies between the nightly build toolchain used by V8 and the released versions that we use from the Linux distribution.

Refs: #62572 (comment)
Refs: https://github.com/ibmruntimes/v8-build/blob/49b55c036da148bd18b546f19b731a0fe92b6a33/bin/v8.sh#L62-L71
Refs: nodejs/build#4265


Opening this as separate from the V8 14.8 update as I want to cherry-pick it across to v26.x. The plan is to update clang and rustc on the RHEL machines and this patch should keep the V8 CI working on v26.x and main during the transition. Without this patch V8 CI would break on Linux on ppc64le and s390x when we update Rust to 1.88 as "adler" has been replaced with "adler2".

@richardlau richardlau added dont-land-on-v22.x PRs that should not land on the v22.x-staging branch and should not be released in v22.x. dont-land-on-v24.x PRs that should not land on the v24.x-staging branch and should not be released in v24.x. dont-land-on-v25.x PRs that should not land on the v25.x-staging branch and should not be released in v25.x. labels May 1, 2026
@nodejs-github-bot nodejs-github-bot added 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 May 1, 2026
Patch the build scripts for V8 when running the V8 CI for Linux on
ppc64le and s390x to workaround inconsistencies between the nightly
build toolchain used by V8 and the released versions that we use from
the Linux distribution.

Signed-off-by: Richard Lau <richard.lau@ibm.com>
@richardlau
Copy link
Copy Markdown
Member Author

Several force pushes to satisfy the sh linter.

Test runs:

Copy link
Copy Markdown
Member

@targos targos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RSLGTM.

I think the build fails because of 7d65f79. I need to remove the tests that use these deleted functions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dont-land-on-v22.x PRs that should not land on the v22.x-staging branch and should not be released in v22.x. dont-land-on-v24.x PRs that should not land on the v24.x-staging branch and should not be released in v24.x. dont-land-on-v25.x PRs that should not land on the v25.x-staging branch and should not be released in v25.x. 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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants