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

deps: update ngtcp2 to 1.1.0 #50808

Closed
wants to merge 2 commits into from
Closed

Conversation

nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot nodejs-github-bot commented Nov 19, 2023

This is an automated update of ngtcp2 to 1.1.0.

@nodejs-github-bot nodejs-github-bot added dependencies Pull requests that update a dependency file. needs-ci PRs that need a full CI run. quic Issues and PRs related to the QUIC implementation / HTTP/3. labels Nov 19, 2023
@marco-ippolito
Copy link
Member

it seems a build failure @jasnell

@jasnell
Copy link
Member

jasnell commented Nov 24, 2023

Yes, we can't rely on the auto updater for this or nghttp3 just yet. I'll be working on this soon.

@nodejs-github-bot nodejs-github-bot changed the title deps: update ngtcp2 to 1.0.1 deps: update ngtcp2 to 1.1.0 Dec 3, 2023
@targos
Copy link
Member

targos commented Dec 23, 2023

Just rebased, fixed the conflict and updated the gyp file.

@targos
Copy link
Member

targos commented Dec 23, 2023

@jasnell The update script and build of ngtcp2 should be fixed. Now our quic code needs to be adapted.

@jasnell
Copy link
Member

jasnell commented Dec 23, 2023

Yep, I'll be working on it this next week :-)

@lpinca
Copy link
Member

lpinca commented Dec 27, 2023

Closing in favor of #51291.

@lpinca lpinca closed this Dec 27, 2023
@lpinca lpinca deleted the actions/tools-update-ngtcp2 branch December 27, 2023 06:24
@lpinca
Copy link
Member

lpinca commented Dec 27, 2023

The changes to the update script should probably be cherry-picked in #51291.

@targos
Copy link
Member

targos commented Dec 27, 2023

I opened #51294 with a generalized version

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. quic Issues and PRs related to the QUIC implementation / HTTP/3.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants