-
Notifications
You must be signed in to change notification settings - Fork 66
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
Incompatible with electron-rebuild / electron-forge #65
Comments
I tried the workaround but that resulted in this:
On Windows 10, node 12.20.1, electron 11 I'd be happy to try any suggestions. I'm not sure what to do about this here. |
@1111mp The error code doesn't say much unfortunately. There may be more details in the log files. Also keep in mind that even though the compile succeeds, there will likely still be issues with it (e.g. placing the output in the wrong folder). I'd still recommend disabling the compile in electron-rebuild. |
When I compile without electron-rebuild, this error still appears. |
Unfortunately those logs still do not contain enough info. If it is an issue with the native module, there should be an error with something like "Cannot find module", which could give a better indication of what the issue is. |
I created the simplest project that can reproduce the problem: https://github.com/1111mp/your-project-name |
That sample project works for me with node 14.0.0 (on Linux, don't have a Windows machine available currently). I ran Do check that you have this file installed: |
It seems like some problem with windows, on Mac it works fine. but it crash on Windows.Testing Project |
Hi guys! I have been able to compile the library in windows using the 5.0.0 version with node 10.17.0. The point is that the library works, that is, the database is created and encrypted, but when it is used in this way, the main window of electron closes without showing any error. You can try this test repo. This occurs both in production and development. When you remove the encryption the project work fine using the database: Using a mac I have tested and it works perfectly. It just stops working using windows Any idea what is happening? @rkistner @1111mp @Christilut? Thanks! Repo: https://github.com/ajmasia/electron-sqlite-playground |
@ajmasia At present I don't know how to solve this problem. It will only happen on the windows platform. |
Thanks @1111mp for your answer. I thought version 4.1.1 only allowed up to electron 8.1.1. I have tested with version 9.4.1 and works fine on both mac and windows, but takes longer to rebuild 😢 |
@ajmasia I am currently using |
Thanks @1111mp, I will check it 😄 |
Hi @rkistner @1111mp I have thoroughly tested The funny thing is that on my local machine it downloads the binaries without a problem 😢 Node version: 12.16.1 Thanks! |
Fixed, was an issue with new Github actions windows image :-) |
Did anyone get this working in a production build with electron-forge? I'm using the
Any ideas? |
Hi everyone,
but I am getting the following error: Cannot find module ’/Documents/app/platforms/electron/build/mac/Application.app/Contents/Resources/app.asar/node_modules/@journeyapps/sqlcipher/lib/binding/napi-v3-darwin-x64/node_sqlite3.node Could someone help me what are the latest versions that I need to get rid of this error? Thanks |
I tried using |
Previously reported in #53 #54 #55 #56 #57 #58 #61 #63. Creating one issue to keep track of this.
There are now workaround instructions here: https://github.com/journeyapps/node-sqlcipher#usage-with-electron-forge--electron-rebuild
It would still be great to find a solution that works without additional configuration.
There are multiple issues with electron-rebuild (which is also used by default with electron-forge) attempting to rebuild this module.
{napi_build_version}
in themodule_path
. See Incorrect folder name when rebuilding "sqlite3" for arm64/x64 architecture electron/rebuild#554 / Cannot rebuild sqlite3 electron/rebuild#546.The workaround is to disable rebuilding of this module with electron-rebuild. However, there are still potential issues:
Disabling N-API v6 support would solve (1) above, but not (2). It would also remove worker_thread support, which is not ideal.
The text was updated successfully, but these errors were encountered: