X Tutup
Skip to content

deps: update ngtcp2 to 1.1.0#50808

Closed
nodejs-github-bot wants to merge 2 commits intomainfrom
actions/tools-update-ngtcp2
Closed

deps: update ngtcp2 to 1.1.0#50808
nodejs-github-bot wants to merge 2 commits intomainfrom
actions/tools-update-ngtcp2

Conversation

@nodejs-github-bot
Copy link
Copy Markdown
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
Copy Markdown
Member

it seems a build failure @jasnell

@jasnell
Copy link
Copy Markdown
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 targos force-pushed the actions/tools-update-ngtcp2 branch from e8e3482 to d115916 Compare December 23, 2023 11:09
@targos
Copy link
Copy Markdown
Member

targos commented Dec 23, 2023

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

@targos targos force-pushed the actions/tools-update-ngtcp2 branch from d115916 to aa87692 Compare December 23, 2023 13:40
@targos
Copy link
Copy Markdown
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
Copy Markdown
Member

jasnell commented Dec 23, 2023

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

@lpinca
Copy link
Copy Markdown
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
Copy Markdown
Member

lpinca commented Dec 27, 2023

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

@targos
Copy link
Copy Markdown
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

X Tutup