-
Notifications
You must be signed in to change notification settings - Fork 584
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
Fix building for "Windows ia32" on main #5673
Comments
Is it related to #4857? |
Most likely not. @RedBeard0531 pointed me to the fact that we're not using |
May I ask if this issue is related to this error:
I found some previous issues with the same error, but all of them were using a version like 10.20 or 11.0.0-rc.2 and the issue is now closed. Btw, I'm using version 11.7.0 |
I'm pretty sure this is caused by nodejs/node-addon-api#1272, which was fixed by nodejs/node-addon-api#1275 / nodejs/node-addon-api@53f7cf1. This was introduced in node-addon-api 5.1.0 and unfortunately only fixed in 6.0.0, so anything using I'll see if I can find a way to force us to either exactly 5.0.0, or at least 6.0.0 |
Pulls in nodejs/node-addon-api@53f7cf1 which fixes nodejs/node-addon-api#1272. Fixes #5673.
I checked and the realm version 11.7.0 uses node-addon-api Sorry if this is not the right place to ask this, maybe I should create a specific issue for this, but I would really appreciate if you have any suggestions about what should I do. |
@rossicler-hostalky please do, that's the only way we can keep track of all the moving parts 🙂 |
no problem, thanks for the quick response |
Pulls in nodejs/node-addon-api@53f7cf1 which fixes nodejs/node-addon-api#1272. Fixes realm#5673.
It seems building for Windows ia32 broke on main.
I start seeing errors on the merge commit from #5650: https://github.com/realm/realm-js/actions/runs/4541450640/jobs/8003632690
These optional dependencies got added back in with https://github.com/realm/realm-js/pull/5652/files#diff-053150b640a7ce75eff69d1a22cae7f0f94ad64ce9a855db544dda0929316519
But on that commit Windows ia32 started failing with:
The diff between the last green build and the first failing commit doesn't show any changes to the bindgen template: e41501e...18140b1
The text was updated successfully, but these errors were encountered: