Conversation
|
@jasnell do those dummy line additions makes sense to you? |
|
We probably need to add a line to the .gitignore like https://github.com/nodejs/node/blob/75ed4eeba518b2bfc999b0fe4ee56d09cc4cdc52/deps/ngtcp2/.gitignore#L8C1-L8C1 |
Not really |
58e309f to
792842f
Compare
|
@marco-ippolito I think something might be wrong with the updater script as the current form of the PR does not make a lot of sense to me. Is that something you could take a look at? |
mhdawson
left a comment
There was a problem hiding this comment.
Marking as changes needed so that it does not land until we figure out if it makes sense or not.
|
to launch update locally, its required to have pkg-config, issue related: ngtcp2/nghttp3#147. Anyways, I downloaded the new release 1.1.0, run configure, deleted deps/ngtcp2/nghttp3/lib and replaced with the lib folder from the release. It matches with the changes created by the script. |
|
I can reproduce the same update locally. I think the lack of changes is due to #51291, which didn't update the version.h |
|
@marco-ippolito, @targos thanks for taking a look, removing my block since it sounds like it makes sense. |
7301dfd to
3701769
Compare
|
Build is failing with |
|
sfparse is now a git submodule in upstream nghttp3: ngtcp2/nghttp3@e755bfb |
|
This needs a rebase. We might as well close this PR so the bot opens a new one |
This is an automated update of nghttp3 to 1.6.0.