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

build: set "clang" in config.gypi in GN build by zcbenz · Pull Request #53004 · nodejs/node · GitHub

/ node Public

build: set "clang" in config.gypi in GN build - #53004

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
zcbenz:gn-config-gypi-clang
May 20, 2024
Merged

build: set "clang" in config.gypi in GN build#53004
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
zcbenz:gn-config-gypi-clang

Conversation

zcbenz commented May 15, 2024

Copy link
Copy Markdown
Contributor

Update the GN build's handling of config.gypi to match the behavior in #52873.

nodejs-github-bot added the tools Issues and PRs related to the tools directory. label May 15, 2024
zcbenz added the request-ci Add this label to start a Jenkins CI on a PR. label May 19, 2024
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label May 19, 2024

Copy link
Copy Markdown
Collaborator

zcbenz added the commit-queue Add this label to land a pull request using GitHub Actions. label May 20, 2024
nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label May 20, 2024
nodejs-github-bot merged commit b1c1faf into nodejs:main May 20, 2024

Copy link
Copy Markdown
Collaborator

Landed in b1c1faf

zcbenz deleted the gn-config-gypi-clang branch May 20, 2024 02:07
targos pushed a commit that referenced this pull request May 21, 2024
PR-URL: #53004
Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
RafaelGSS mentioned this pull request Jun 7, 2024
soophoo pushed a commit to soophoo/node that referenced this pull request Jun 20, 2024
PR-URL: nodejs#53004
Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
marco-ippolito pushed a commit that referenced this pull request Jul 19, 2024
PR-URL: #53004
Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
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

tools Issues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants


Back | FazBrowse Home | New Git URL