-
-
Notifications
You must be signed in to change notification settings - Fork 28
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
[Ezra bible app] error while building sqlite3 module #23
Comments
Yes, Electron updates broken this package. I've made several attempts to fix it and always come up short. I see today that 1.2.0 is out, I'll take another stab at getting it up to date. Thanks for the report. |
I was able to have something that compiles when adding this in the PKGBUILD before the The error I get in the console is:
|
I just had a pass at this and this is an absolute fiasco. Probably the best thing we could do for now is setup a -bin package variant to install some upstream built variant complete with a bundled version of Electron and everything else. Getting a source build to work properly is going to take some more work, it's just so scrambled... |
Yeah, it would be way easier indeed. Electron packages are apparently famously hard to package. I have no experience in nodejs let alone electron, but I can try to make a PR upstream building a binary in the CI. Thanks for taking the time to look into it! |
I just tried setting up a binary package (now at ezra-bible-app-bin) but it doesn't work because the latest Fedora build at least is linked against a version of ICU older than what Arch Linux has. The next thing to do is probably experiment with the other possible sources (Debian builds perhaps) and see if we can find one that is a match for the various libraries on Arch. Baring that we're back to trying to figure out a source build. Not impossible just a wrestling match. |
I mentioned this sqlite3 > 5.0.0 oddity in an upstream issue ezra-bible-app/ezra-bible-app#436. Also see ezra-bible-app/ezra-bible-app#23 about upstream embedding sqlite3 in the first place. |
@sweenu I've applied your little trick to lock down the sqlite3 version and get a build ... but it's broken for me in the same way you describe at runtime. I just have a dotty spinner. |
This seems to be caused by changes in newer Electron versions. I just created an issue to plan bumping Electron in Ezra Bible App to a newer version. |
Note that ezra-bible-app/ezra-bible-app#437 is done now. |
Thanks, that's progress. With that in place the ezra-bible-app-git package (using the VCS package version to test things in between releases) now builds and installs just fine. We're still hung up on whatever the runtime issue(s) are that make it hang with a spinner on launch. From the console output I see: $ ezra-bible-app
Using settings file /home/caleb/.config/ezra-bible-app/config.json
Detected old InstallMgr directory installMgr.
Deleting previously used InstallMgr dir!
Using default database dir /home/caleb/.config/ezra-bible-app for database access!
Initializing database at /home/caleb/.config/ezra-bible-app
Database not yet existing in user directory!
Copying database from previously used application directory /home/caleb/.config/ezra-project/ezra.sqlite.
Error occurred in handler for 'initIpc': Error: Please install sqlite3 package manually
at ConnectionManager._loadDialectModule (/usr/lib/ezra-bible-app/resources/app.asar/node_modules/sequelize/lib/dialects/abstract/connection-manager.js:88:15)
at new ConnectionManager (/usr/lib/ezra-bible-app/resources/app.asar/node_modules/sequelize/lib/dialects/sqlite/connection-manager.js:22:21)
at new SqliteDialect (/usr/lib/ezra-bible-app/resources/app.asar/node_modules/sequelize/lib/dialects/sqlite/index.js:14:30)
at new Sequelize (/usr/lib/ezra-bible-app/resources/app.asar/node_modules/sequelize/lib/sequelize.js:334:20)
at DbHelper.getSequelize (/usr/lib/ezra-bible-app/resources/app.asar/app/backend/database/db_helper.js:114:21)
at DbHelper.migrateDatabase (/usr/lib/ezra-bible-app/resources/app.asar/app/backend/database/db_helper.js:119:26)
at DbHelper.initDatabase (/usr/lib/ezra-bible-app/resources/app.asar/app/backend/database/db_helper.js:42:16)
at IpcDbHandler.initDatabase (/usr/lib/ezra-bible-app/resources/app.asar/app/backend/ipc/ipc_db_handler.js:52:20)
at IPC.initDatabase (/usr/lib/ezra-bible-app/resources/app.asar/app/backend/ipc/ipc.js:85:24)
at IPC.init (/usr/lib/ezra-bible-app/resources/app.asar/app/backend/ipc/ipc.js:78:20)
at /usr/lib/ezra-bible-app/resources/app.asar/main.js:121:17
at electron/js2c/browser_init.js:197:579
at Object.<anonymous> (electron/js2c/browser_init.js:161:9782)
at Object.emit (events.js:315:20) Note this is without any hackery to change the sqlite3 version or replace it with a system-installed library. A system |
I believe the runtime issues are caused by some sort of incompatibility between the compiled nodejs module and the Electron binary. I have seen this before. Will have a look. |
I tried to get a local environment with Arch Linux running (first Docker, then VirtualBox) - but to no avail. |
@alerque This is the Of course you may have to adjust the Electron version based on what's used in your case. |
@alerque Any feedback here? |
I'm on the road right now and away from my usual build systems and have very limited time for FOSS until I stop traveling in December. |
@alerque Any news on this? :) |
Hello, thanks for the aur package. It does not work however for me right now, failing with these errors:
I have the latest versions of electron (13.2.2) and nodejs (v16.8.0), this might be the problem I suppose.
The text was updated successfully, but these errors were encountered: