Skip to content

build: remove deleted V8 headers from metadata inputs - #66531

Open
inoway46 wants to merge 1 commit into
nodejs:mainfrom
inoway46:fix-v8-postmortem-inputs
Open

inoway46 wants to merge 1 commit into
nodejs:mainfrom
inoway46:fix-v8-postmortem-inputs

Conversation

@inoway46

@inoway46 inoway46 commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

Remove two deleted inline headers from the postmortem-metadata inputs. These stale references cause Ninja builds to fail after the V8 15.2 update:

ninja: error: '../../deps/v8/src/objects/primitive-heap-object-inl.h', needed by 'obj/deps/v8/src/snapshot/embedded/mksnapshot.embedded-file-writer.o', missing and no known rule to make it

Verified that the Ninja dry-run and the postmortem-metadata target complete after the change.

Refs: #65161

Remove two inline headers deleted by the V8 15.2 update from the
postmortem-metadata inputs. Their remaining references prevent Ninja
from building the target.

Assisted-by: Codex
Signed-off-by: inoway46 <inoueyuya416@gmail.com>
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

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

@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 Oct 5, 2026
@inoway46
inoway46 marked this pull request as ready for review October 5, 2026 13:46
@aduh95 aduh95 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-v26.x PRs that should not land on the v26.x-staging branch and should not be released in v26.x. labels Oct 5, 2026
@inoway46

inoway46 commented Oct 5, 2026

Copy link
Copy Markdown
Contributor Author

The full Ninja build completed successfully locally on macOS x64.

[1786/1786] LINK cctest, POSTBUILDS

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

3 participants