nodejs / node

Node.js JavaScript runtime ✨🐢🚀✨
https://nodejs.org
Other
104.75k stars 28.3k forks source link

deps: update nghttp2 to 1.62.1 #52966

Closed nodejs-github-bot closed 1 week ago

nodejs-github-bot commented 1 month ago

This is an automated update of nghttp2 to 1.62.1.

nodejs-github-bot commented 1 month ago

Review requested:

targos commented 1 month ago

Don't land because the new release requires C++20

nodejs-github-bot commented 1 month ago

CI: https://ci.nodejs.org/job/node-test-pull-request/59202/

nodejs-github-bot commented 1 month ago

CI: https://ci.nodejs.org/job/node-test-pull-request/59203/

nodejs-github-bot commented 2 weeks ago

CI: https://ci.nodejs.org/job/node-test-pull-request/59421/

nodejs-github-bot commented 1 week ago

CI: https://ci.nodejs.org/job/node-test-pull-request/59672/

nodejs-github-bot commented 1 week ago

CI: https://ci.nodejs.org/job/node-test-pull-request/59674/

nodejs-github-bot commented 1 week ago

CI: https://ci.nodejs.org/job/node-test-pull-request/59675/

nodejs-github-bot commented 1 week ago

CI: https://ci.nodejs.org/job/node-test-pull-request/59676/

richardlau commented 1 week ago

Don't land because the new release requires C++20

We're not seeing it on our CI but there are reports in upstream nghttp2 re. the new C++20 requirement and compile failures on certain gcc versions/Linux distros: https://github.com/nghttp2/nghttp2/issues/2194

nodejs-github-bot commented 1 week ago

Landed in 1aab854fa8192bba64dbbe5cbfb5444b371b4aa1...4648185867e0e60ed888c7f60cbd8d923245d4fc