Skip to content
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

Builds produced for electron 11 are broken #275

Closed
Julusian opened this issue Apr 14, 2021 · 3 comments
Closed

Builds produced for electron 11 are broken #275

Julusian opened this issue Apr 14, 2021 · 3 comments

Comments

@Julusian
Copy link

This is a snippet of a discovery found in serialport/node-serialport#2191 (comment)

I have raised an issue (electron/node-abi#103) with node-abi to investigate a fix, but I don't know if they will be willing to do anything

When building for electron 11, prebuild is using the headers for v11.0.0-beta.11. However due to some changes introduced in v11.0.0-beta.12, these binaries can be incompatible with other versions of electron 11.
I have confirmed this in that library by trying to run builds produced directly from node-gyp using both v11.0.0-beta.11 and v11.0.0-beta.12 headers in various versions of electron 11. I also manually applied this patch onto my cached v11.0.0-beta.11 headers to confirm it is the snippet at fault.

The fix is simply to use a different version of the headers than node-abi says. Hopefully this is a one-off and wont be a recurring issue in future electron versions

@lovell
Copy link
Member

lovell commented Sep 6, 2023

@Julusian Is this still an issue, or can this now be closed?

@Julusian
Copy link
Author

I have no idea. I now avoid any non node-api native library, to avoid issues such as this.

@lovell
Copy link
Member

lovell commented Sep 13, 2023

@Julusian Thank you, it looks like serialport migrated to Node-API in late 2021 / early 2022 as part of serialport/node-serialport#2305 so I'll close.

@lovell lovell closed this as completed Sep 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants