Skip to content

build: add host toolset to perfetto_sdk - #64751

Open
islandryu wants to merge 1 commit into
nodejs:mainfrom
islandryu:build-perfetto-host-toolset
Open

build: add host toolset to perfetto_sdk#64751
islandryu wants to merge 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

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

Labels

commit-queue-failed An error occurred while landing this pull request using GitHub Actions. 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