Skip to content

Update to node 24.1.0 #66

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

Open
wants to merge 1 commit into
base: node-23
Choose a base branch
from
Open

Conversation

bnoordhuis
Copy link
Collaborator

This brings in a new major version of v8 and Various tweaks are needed:

  • upgrade the CI images to get a toolchain capable of compiling V8

  • upgrade the macos image to macos-15-xlarge because the non-xlarge runner does not have enough memory

  • compile without debug info on the macos runner, otherwise it runs out of free disk space

  • patch a header file to pacify clang

  • link in V8's new dependencies abseil, highway and simdutf

  • disable ccache because it does not seem to play well with V8 anymore and is not otherwise useful for building on CI (hit rate: 0%)

This brings in a new major version of v8 and Various tweaks are needed:

- upgrade the CI images to get a toolchain capable of compiling V8

- upgrade the macos image to macos-15-xlarge because the non-xlarge
  runner does not have enough memory

- compile without debug info on the macos runner, otherwise it runs out
  of free disk space

- patch a header file to pacify clang

- link in V8's new dependencies abseil, highway and simdutf

- disable ccache because it does not seem to play well with V8 anymore
  and is not otherwise useful for building on CI (hit rate: 0%)
@bnoordhuis bnoordhuis force-pushed the node-24 branch 2 times, most recently from df1a77e to 6101402 Compare June 12, 2025 12:09
@bnoordhuis bnoordhuis marked this pull request as ready for review June 12, 2025 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant