-
-
Notifications
You must be signed in to change notification settings - Fork 399
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
Add support for electron v26
prebuilds
#1042
Conversation
(I just want to say thank you!) |
is it possible to automate this? grab all major versions of the electron npm package -> build for latest x versions? Electron only supports the latest 3 major versions. so maybe for older projects support the latest 5. |
Possible but not recommended. The whole dependency tree responsible for building is way too unreliable for automation to be stable. We always say a quick prayer before building for the latest Electron release.
The prebuild support is determined by whatever Node versions are being actively supported. That's why we still build for Electron v16, which ships with embedded Node v16 (still in active maintenance). |
@m4heshd alrighty. i just asked it for the electron part, because you are building it for very old electron versions, which are not supported by electron for a long time. but i am fine with your answer. thank you very much :) |
That's exactly what I explained. Even though Electron drops support for these versions pretty quickly, they still ship with actively maintained Node versions (I'm assuming here that you already know about how Electron carries its own copy of Node as a library). If you run Electron v16 with |
Yes (And i got in trouble because of that (i am looking at you openssl vs boringssl ^^), but...
this not. Thanks for further explanation 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Just tried it with electron forge (vite template, [email protected] and [email protected]) and it's not working, everything works when I downgrade to [email protected] and [email protected], anyone else having problems? |
Do you use the correct nodes version in your project? Electron 26 is using node 18.16.1 |
@ferminc It's possible that your native modules aren't properly rebuilt for the newer version of Electron you're using. Since you're using |
i can confirm everything is working with electron-builer, electron 26 and latest better-sqlite |
Did you encounter electron/electron#39600 in electron 26? |
This is the error I'm getting
So, maybe there's something going on with forge, I haven't tried using electron-builder |
I've also encountered this issue. Was your issue resolved? How it resolved? |
They haven't release a fix yet, but you can fix it yourself rebuilding manually: #1044 (comment) you'll need to have visual studio and python installed |
I got the same issue with @capacitor-community/sqlite which use
my node version is v18.17.1 |
Stable electron
v26
just released. Prebuilds were tested here.