Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[v18.x] deps: update nghttp2 to 1.57.0 #50151

Closed
wants to merge 6 commits into from

Conversation

richardlau
Copy link
Member

This is a backport to Node.js 18 of:

Node.js 18.18.0 went out with the nghttp2 1.55.0 update (#48746) but without first applying the nghttp2 1.53.0 update (#47997). #47997 was blocked because it depended on part of #46965.

I've reverted the ngthttp2 1.55.0 commit on v18.x-staging, and then re-cherry-picked the nghttp2 update commits from main, fixing up conflicts along the way. For #47997 that involved manually fixing up nghttp2.gyp.

Subsequent updates had conflicts in doc/contributing/maintaining/maintaining-dependencies.md because we haven't backported #48081 -- for these I've undone the changes to the maintaining-dependencies.md file via:

git checkout --ours HEAD -- doc/contributing/maintaining/maintaining-dependencies.md

richardlau and others added 6 commits October 11, 2023 14:30
PR-URL: nodejs#47997
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
PR-URL: nodejs#48746
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
PR-URL: nodejs#48790
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Yongsheng Zhang <[email protected]>
Reviewed-By: Stephen Belanger <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
PR-URL: nodejs#50121
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/gyp
  • @nodejs/http2
  • @nodejs/net

@nodejs-github-bot nodejs-github-bot added build Issues and PRs related to build files or the CI. http2 Issues or PRs related to the http2 subsystem. needs-ci PRs that need a full CI run. v18.x Issues that can be reproduced on v18.x or PRs targeting the v18.x-staging branch. labels Oct 11, 2023
@richardlau richardlau added request-ci Add this label to start a Jenkins CI on a PR. and removed build Issues and PRs related to build files or the CI. http2 Issues or PRs related to the http2 subsystem. needs-ci PRs that need a full CI run. v18.x Issues that can be reproduced on v18.x or PRs targeting the v18.x-staging branch. labels Oct 11, 2023
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Oct 11, 2023
@nodejs-github-bot
Copy link
Collaborator

Copy link
Member

@RafaelGSS RafaelGSS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RSLGTM

@nodejs-github-bot
Copy link
Collaborator

@richardlau
Copy link
Member Author

Landed in b2b3118...a792bbc.

@richardlau richardlau closed this Oct 12, 2023
richardlau added a commit that referenced this pull request Oct 12, 2023
This reverts commit c8da8c8.

PR-URL: #50151
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
richardlau pushed a commit that referenced this pull request Oct 12, 2023
PR-URL: #47997
Backport-PR-URL: #50151
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
richardlau pushed a commit that referenced this pull request Oct 12, 2023
PR-URL: #48746
Backport-PR-URL: #50151
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
richardlau pushed a commit that referenced this pull request Oct 12, 2023
PR-URL: #48790
Backport-PR-URL: #50151
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Yongsheng Zhang <[email protected]>
Reviewed-By: Stephen Belanger <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
richardlau pushed a commit that referenced this pull request Oct 12, 2023
PR-URL: #49582
Backport-PR-URL: #50151
Reviewed-By: Luigi Pinca <[email protected]>
richardlau pushed a commit that referenced this pull request Oct 12, 2023
PR-URL: #50121
Backport-PR-URL: #50151
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
@richardlau richardlau deleted the v18.x-nghttp2 branch October 12, 2023 14:03
@richardlau richardlau added the v18.x Issues that can be reproduced on v18.x or PRs targeting the v18.x-staging branch. label Oct 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v18.x Issues that can be reproduced on v18.x or PRs targeting the v18.x-staging branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants