-
Notifications
You must be signed in to change notification settings - Fork 29.8k
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
Breaking ABI on an existing release line by bumping NODE_MODULE_VERSION #49875
Comments
cc @nodejs/tsc |
Given that v16 is out of support, I'm +1 with the ABI breakage before v20 hits LTS. |
I do believe this is going to break prebuilt native addons if they are not using Node-API. I don't think we should be doing this unless we have something that is In terms of v20, developers to have worked in advance to create their prebuilt addons for 20.x are likely to be unimpressed if we break then just a few weeks before the LTS switch. |
@joyeecheung have you got a tracking issue for the breaking of Jest? What % of users are affected? |
So I just realized that since we can "add whatever we want to the v8 fork on v20.x", basically, we can use some Node.js-only extensions to the V8 API there to avoid the ABI breakages completely. Not sure if we should still keep the issue open though, as ABI breakages might still come up again, maybe we need to update the doc about whether this is feasible or not.. |
According to
node/doc/contributing/releases.md
Line 1210 in f16f41c
Now the time comes when we may need to break the ABI in order to backport to v20.x a series of memory issue fixes that are blocking people (in particular jest users) from upgrading away from v16.x: #49874 .EDIT: turns out that we found a way to avoid the ABI breakage.
This issue is used to figure out
The text was updated successfully, but these errors were encountered: