Skip to content

build: add host toolset to perfetto_sdk - #64751

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
islandryu:build-perfetto-host-toolset
Aug 4, 2026
Merged

nodejs-github-bot merged 1 commit into
nodejs:mainfrom
islandryu:build-perfetto-host-toolset

Conversation

@islandryu

Copy link
Copy Markdown
Member

./configure -C --with-perfetto fails GYP because v8_init#host (mksnapshot) needs perfetto_sdk#host, but perfetto_sdk never declared the host toolset.

Add 'toolsets': ['host', 'target'], matching temporal_capi and libuv.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/gyp
  • @nodejs/security-wg

@nodejs-github-bot nodejs-github-bot added dependencies PRs that add, update, or configure Node.js dependencies. needs-ci PRs that need a full CI run. labels Jul 26, 2026
@islandryu
islandryu force-pushed the build-perfetto-host-toolset branch from 399790c to ad4369b Compare July 26, 2026 05:37
@islandryu islandryu added the request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. label Jul 26, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. label Jul 26, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@islandryu islandryu added the commit-queue PRs queued for automated landing through the Commit Queue. label Jul 30, 2026
@nodejs-github-bot nodejs-github-bot added commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. and removed commit-queue PRs queued for automated landing through the Commit Queue. labels Jul 30, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/64751
✔  Done loading data for nodejs/node/pull/64751
----------------------------------- PR info ------------------------------------
Title      build: add host toolset to perfetto_sdk (#64751)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     islandryu:build-perfetto-host-toolset -> nodejs:main
Labels     needs-ci, dependencies
Commits    1
 - build: add host toolset to perfetto_sdk
Committers 1
 - islandryu <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/64751
Reviewed-By: Aviv Keller <[email protected]>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/64751
Reviewed-By: Aviv Keller <[email protected]>
--------------------------------------------------------------------------------
   ℹ  This PR was created on Sun, 26 Jul 2026 05:08:22 GMT
   ✔  Approvals: 1
   ✔  - Aviv Keller (@avivkeller): https://github.com/nodejs/node/pull/64751#pullrequestreview-4781136782
   ✘  This PR needs to wait 73 more hours to land (or 0 minutes if there is one more approval)
   ✔  Last GitHub CI successful
   ℹ  Last Full PR CI on 2026-07-29T14:52:12Z: https://ci.nodejs.org/job/node-test-pull-request/75291/
- Querying data for job/node-test-pull-request/75291/
✔  Build data downloaded
   ✔  Last Jenkins CI successful
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/30513852748

@islandryu islandryu added commit-queue PRs queued for automated landing through the Commit Queue. and removed commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. labels Aug 4, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Aug 4, 2026
@nodejs-github-bot
nodejs-github-bot merged commit 427d2e1 into nodejs:main Aug 4, 2026
77 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 427d2e1

aduh95 pushed a commit that referenced this pull request Aug 13, 2026
Signed-off-by: islandryu <[email protected]>
PR-URL: #64751
Reviewed-By: Aviv Keller <[email protected]>
aduh95 pushed a commit that referenced this pull request Aug 25, 2026
Signed-off-by: islandryu <[email protected]>
PR-URL: #64751
Reviewed-By: Aviv Keller <[email protected]>
@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. labels Sep 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies PRs that add, update, or configure Node.js dependencies. 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. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants