Skip to content

build: build codecache and snapshot with libnode#63626

Merged
nodejs-github-bot merged 5 commits into
nodejs:mainfrom
legendecas:static-intermediate
Jun 2, 2026
Merged

build: build codecache and snapshot with libnode#63626
nodejs-github-bot merged 5 commits into
nodejs:mainfrom
legendecas:static-intermediate

Conversation

@legendecas
Copy link
Copy Markdown
Member

@legendecas legendecas commented May 28, 2026

This refactors the node and libnode targets, splitting the node_mksnapshot action from the node exe target, and adding a (always) static node_base target, allowing libnode to depend on node_mksnapshot exe, enabling the node startup snapshot when building with flag --shared.

The new dependency graph would be like:

node (executable)
└── libnode (static or shared)
    ├── node_base (static, whole-archived by `node` when not `--shared`)
    │   ├── node_js2c#host
    │   └── <all C++ sources + js2c generated source>
    ├── node_mksnapshot (executable, build-time)
    │   └── node_base
    └── node_snapshot.cc (generated by `node_mksnapshot`)
        OR node_snapshot_stub.cc

cctest, fuzz_*, embedtest, etc.
└── libnode

@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/gyp

@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. labels May 28, 2026
@legendecas legendecas force-pushed the static-intermediate branch 2 times, most recently from adf769d to fac4309 Compare May 28, 2026 20:16
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
@legendecas legendecas force-pushed the static-intermediate branch from fac4309 to e880392 Compare May 28, 2026 20:17
Copy link
Copy Markdown
Member

@joyeecheung joyeecheung left a comment

Choose a reason for hiding this comment

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

Thanks. LGTM if CI is happy!

Comment thread configure.py
@joyeecheung joyeecheung added the request-ci Add this label to start a Jenkins CI on a PR. label May 28, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label May 28, 2026
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@legendecas legendecas added the request-ci Add this label to start a Jenkins CI on a PR. label Jun 1, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 1, 2026
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@legendecas legendecas added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Jun 2, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 2, 2026
@nodejs-github-bot nodejs-github-bot merged commit 3d35dd8 into nodejs:main Jun 2, 2026
75 checks passed
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

Landed in 3d35dd8

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

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants