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

build: add host toolset to perfetto_sdk#64751
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 Pull requests that update a dependency file. needs-ci PRs that need a full CI run. labels Jul 26, 2026
Signed-off-by: islandryu <shimaryuhei@gmail.com>
@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. label Jul 26, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. 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 Add this label to land a pull request using GitHub Actions. label Jul 30, 2026
@nodejs-github-bot nodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. 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 <shimaryuhei@gmail.com>
PR-URL: https://github.com/nodejs/node/pull/64751
Reviewed-By: Aviv Keller <me@aviv.sh>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/64751
Reviewed-By: Aviv Keller <me@aviv.sh>
--------------------------------------------------------------------------------
   ℹ  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 Add this label to land a pull request using GitHub Actions. and removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. labels Aug 4, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. 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 <shimaryuhei@gmail.com>
PR-URL: #64751
Reviewed-By: Aviv Keller <me@aviv.sh>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants