Skip to content

build: def NODE_USE_NODE_CODE_CACHE only used in node_mksnapshot#63588

Open
legendecas wants to merge 1 commit into
nodejs:mainfrom
legendecas:node_use_node_code_cache
Open

build: def NODE_USE_NODE_CODE_CACHE only used in node_mksnapshot#63588
legendecas wants to merge 1 commit into
nodejs:mainfrom
legendecas:node_use_node_code_cache

Conversation

@legendecas
Copy link
Copy Markdown
Member

NODE_USE_NODE_CODE_CACHE is only used in node_mksnapshot.cc. It is unnecessary to define it for target 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 26, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
@legendecas legendecas force-pushed the node_use_node_code_cache branch from dc8150a to 4577d5e Compare May 26, 2026 18:32
@legendecas legendecas added request-ci Add this label to start a Jenkins CI on a PR. author ready PRs that have at least one approval, no pending requests for changes, and a CI started. labels May 26, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label May 26, 2026
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

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