-
Notifications
You must be signed in to change notification settings - Fork 52
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
was compiled against a different Node.js version 64 vs 72 #41
Comments
I have exact the same issue in an Electron 5.0 project. In my condition, the postinstall script calls electron-builder to rebuilds binarys for electron, when I try to test this function without electron, the error occurs. You can run
NOTE: Needs to run @jduncanator pls add multi target binary feature like sqlite3 do, which will generate binary in 'electron-v5.0-win32-x64' and 'node-v64-win32-x64', then we don't have to care about platforms. |
I'll take a look into this problem. Be aware that diskusage targets multiple platforms, so using the provided "mokey patch" to index.js will only resolve the problem when running on Windows. I'll investigate a more cross platform friendly fix. |
I'm still not sure exactly what steps reproduce this issue. Can you provide some more information into how you ran into this? |
Thank you for concerning about this issue. I reproduced it with a clean new vue-electron project, here are the steps:
Here is my test project which reproduces the issue: The sqlite3 module provides a example for resolving this, it keeps native modules for both os and electron, and auto select them in runtime. |
Great, thank you for the detailed steps. I'll take a look at this today and hopefully push a fix! |
@jduncanator any update on that? |
Still broken it seems. |
Broken for me on node 13.0.1.. need to search for some other package |
Broken for me on Node v12.18.1 |
|
Hello, I got this issue this morning after installing...
node version 10.16.0
npm version 6.9.0
node-gyp version 4.0.0
anyone having this issue?
The text was updated successfully, but these errors were encountered: