You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Pretty soon we'll need to update the zlib version (see nodejs/security-wg#824). I've tried to update it locally following the maintaining-zlib document, but things get breaking (it seems arch issues -- I'm on Linux).
AFAIK the document is up-to-date, it's just that upstream made a lot of changes to the build config and this line is all about it:
It may be necessary to update deps/zlib/zlib.gyp if any significant changes have occurred upstream.
There have been multiple attempts to update zlib without success so far (I think it's mostly a question of time and C/build knowledge): #33044, #41745, #42570, #42571
My branch for #41745 went quite far, but there were performance concerns which I'm unable to handle.
Activity
Trott commentedon Aug 16, 2022
@nodejs/zlib
targos commentedon Aug 17, 2022
AFAIK the document is up-to-date, it's just that upstream made a lot of changes to the build config and this line is all about it:
There have been multiple attempts to update zlib without success so far (I think it's mostly a question of time and C/build knowledge): #33044, #41745, #42570, #42571
My branch for #41745 went quite far, but there were performance concerns which I'm unable to handle.
RafaelGSS commentedon Aug 17, 2022
Ok, thanks!
I'll look at it and work soon as I have time. I'll keep this issue posted, probably we would need to update the document as well.
Neustradamus commentedon Oct 15, 2022
To follow this ticket :)
lpinca commentedon Nov 17, 2022
I'm closing this as #45387 landed. Please comment or reopen if needed.