-
-
Notifications
You must be signed in to change notification settings - Fork 93
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
"App threw an error during load" with latest electron 4.0.0 and electron-edge-js 10.11.0 #32
Comments
I have the same issue. |
Also experiencing the same issue. Electron 4.0.0 Node 10.11.0. |
The issue appears to be with the way 4.0.0 packages Node.js. Previous versions had |
@agracio Has there been any progress on this? |
Probably related to wrong ABI in electron 3 which causes prebuild to fail |
"However, in Electron 4.x and higher, the symbols needed by native modules are exported by electron.exe, and there is no node.dll in Electron 4.x." https://electronjs.org/docs/tutorial/using-native-node-modules#troubleshooting |
Continuing with @kirilpopov linked documentation is the required change:
Furthermore, this is listed in 4.0's breaking changes:
Therefore, it appears we need to change this setting. |
Tried rebuilding (using electron-rebuild) after updating win_delay_load_hook to true in bindings.gyp file. Now electron silently dies after start :( - with this in the windows event viewer
|
I can confirm the results @kirilpopov found. It looked like it was possibly an issue with the C++ Redistributable dlls beside the Here is the verbose output from building. I've tried to bold the flags referenced here: https://electronjs.org/docs/tutorial/using-native-node-modules#troubleshooting. I can post full output if needed... Link: |
I tried rebuilding the code against Electron 4.0.4 (also modified the binding.gyp file mentioned above) ".\node_modules.bin\electron-rebuild.cmd 4.0.4" and the generated edge_coreclr.node works fine. With edge_nativecrl.node it produces the following error (at the require line):
|
Some progress: the problem is with the win_delay_load_hook.cc file. Somewhy on module load the symbols are not loaded. When I remove this file and put all the code in the edge.cpp the sample code runs without any error. |
OK, the solution seems very simple:
|
Works on my side. A possible change in electron-edge-js would be to keep 'win_delay_load_hook' to false and add new file that registers the hook, exactly the same as win_delay_load_hook.cc but with the extra pragma |
Yes, but it is also important to add |
Closing the issue as it works fine with 10.11.1 on my side now. |
Hey there,
upgraded to electron 3.0.13 and electron-edge-js 10.11.0 lately and everything was running fine (app could be build, started and .Net integration was working).
After the stable release of electron 4.0.0 I now tried to upgrade it but my electron app fails on start with the following error:
I already tried a clean install by removing the node_module folders.
Here on github it looks like electron-edge-js should already be ready for electron 4.x.x releases but seems its not or do I miss something in my setup?
best regards
Max
The text was updated successfully, but these errors were encountered: