FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

build: add host toolset to perfetto_sdk by islandryu · Pull Request #64751 · nodejs/node · GitHub

/ node Public

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

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.

Copy link
Copy Markdown
Collaborator

Review requested:

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

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 force-pushed the build-perfetto-host-toolset branch from 399790c to ad4369b Compare July 26, 2026 05:37
islandryu added the request-ci Add this label to start a Jenkins CI on a PR. label Jul 26, 2026
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 26, 2026

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

islandryu added the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 30, 2026
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

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 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 removed the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 4, 2026
nodejs-github-bot merged commit 427d2e1 into nodejs:main Aug 4, 2026
77 checks passed

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>
aduh95 pushed a commit that referenced this pull request Aug 25, 2026
Signed-off-by: islandryu <shimaryuhei@gmail.com>
PR-URL: #64751
Reviewed-By: Aviv Keller <me@aviv.sh>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL