Skip to content

Rolling to V8 15.2.124.1 - #2037

Merged
bartlomieju merged 15 commits into
denoland:mainfrom
nathanwhit:autoroll
Aug 1, 2026
Merged

Rolling to V8 15.2.124.1#2037
bartlomieju merged 15 commits into
denoland:mainfrom
nathanwhit:autoroll

Conversation

@nathanwhit

Copy link
Copy Markdown
Member

Rolls the V8 submodule and README to V8 15.2.124.1.

This also fixes fork handling in tools/auto_update_v8.ts. The updater previously used origin for checkout and push, but hardcoded denoland:autoroll as the pull-request head. In a fork checkout, that combined the locally calculated title with an unrelated upstream branch.

The updater now:

  • resolves the denoland/rusty_v8 remote independently for the main base;
  • derives the pull-request head owner from the actual push remote;
  • uses explicit repository, base, and head arguments for gh;
  • preserves and stages manual updater changes across its checkout; and
  • prints command stderr when a subprocess fails.

Validation:

  • deno fmt --check tools/auto_update_v8.ts
  • deno lint tools/auto_update_v8.ts
  • deno check tools/auto_update_v8.ts
  • verified fork and GitHub Actions remote selection
  • verified c4ca1eccb90c5464d826b7713dc27178a53b0cfe is the current 15.2-lkgr-denoland head and reports V8 15.2.124.1
  • verified V8 15.2 changes none of the dependency gitlinks pinned by upstream main

@nathanwhit
nathanwhit marked this pull request as ready for review July 29, 2026 21:08

@ry ry left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@bartlomieju bartlomieju left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Converting my three earlier top-level comments into inline ones anchored to the relevant code, plus two smaller notes that were buried at the end of the crdtp comment. Deleting the originals.

Comment thread src/binding.cc Outdated
Comment thread tools/auto_update_v8.ts Outdated
Comment thread src/crdtp_binding.cc Outdated
Comment thread src/crdtp.rs Outdated
Comment thread tests/test_api.rs
@bartlomieju
bartlomieju merged commit 4da6147 into denoland:main Aug 1, 2026
40 checks passed
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.

3 participants