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

deps: c-ares: cherry-pick 64df444 by zcbenz · Pull Request #53706 · nodejs/node · GitHub

/ node Public

deps: c-ares: cherry-pick 64df444 - #53706

Closed
zcbenz wants to merge 1 commit into
nodejs:mainfrom
zcbenz:fix-cares-win-build
Closed

deps: c-ares: cherry-pick 64df444#53706
zcbenz wants to merge 1 commit into
nodejs:mainfrom
zcbenz:fix-cares-win-build

Conversation

zcbenz commented Jul 3, 2024

Copy link
Copy Markdown
Contributor

Backport c-ares/c-ares#802 which fixes compilation error when building for Windows with UNICODE defined.

Backport c-ares/c-ares#802 which fixes
compilation error when building for Windows with UNICODE defined.

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/net
  • @nodejs/security-wg

nodejs-github-bot added cares Issues and PRs related to the c-ares dependency or the cares_wrap binding. dependencies Pull requests that update a dependency file. needs-ci PRs that need a full CI run. labels Jul 3, 2024
zcbenz added request-ci Add this label to start a Jenkins CI on a PR. and removed cares Issues and PRs related to the c-ares dependency or the cares_wrap binding. needs-ci PRs that need a full CI run. dependencies Pull requests that update a dependency file. labels Jul 3, 2024
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 3, 2024

Copy link
Copy Markdown
Collaborator

zcbenz added the request-ci Add this label to start a Jenkins CI on a PR. label Jul 4, 2024
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 4, 2024

Copy link
Copy Markdown
Collaborator

lpinca commented Jul 4, 2024

Copy link
Copy Markdown
Member

A new version of c-ares is already available. I think it is better to upgrade to that version instead of cherry picking this commit.

zcbenz commented Jul 4, 2024

Copy link
Copy Markdown
Contributor Author

I’m good updating to the latest version instead.

lpinca commented Jul 4, 2024

Copy link
Copy Markdown
Member

See #53722.

zcbenz closed this Jul 5, 2024
zcbenz deleted the fix-cares-win-build branch July 5, 2024 00:30
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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL