[PR #3220] [MERGED] :electron: Fix "Unknown problem opening <local actual file>" on Electron #4742

Closed
opened 2026-02-28 21:00:33 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/3220
Author: @MikesGlitch
Created: 8/8/2024
Status: Merged
Merged: 8/12/2024
Merged by: @MikesGlitch

Base: masterHead: electron/testing


📝 Commits (10+)

📊 Changes

4 files changed (+52 additions, -14 deletions)

View changed files

📝 bin/package-electron (+2 -4)
packages/desktop-electron/beforePackHook.ts (+36 -0)
📝 packages/desktop-electron/package.json (+7 -10)
upcoming-release-notes/3220.md (+7 -0)

📄 Description

Fixes:

Root cause:

  • better-sqlite3 was being built incorrectly.

Solution:

  • Move away from "Universal" builds for now, build for x64 and arm64.
  • Before packing the app, detect which arch we're building for and build better-sqlite3 manually for that arch

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/actualbudget/actual/pull/3220 **Author:** [@MikesGlitch](https://github.com/MikesGlitch) **Created:** 8/8/2024 **Status:** ✅ Merged **Merged:** 8/12/2024 **Merged by:** [@MikesGlitch](https://github.com/MikesGlitch) **Base:** `master` ← **Head:** `electron/testing` --- ### 📝 Commits (10+) - [`35ea4b7`](https://github.com/actualbudget/actual/commit/35ea4b70afb5ef0d8bca34c20b5f7b6505b3118f) add electron logging to main browser process console - [`d158198`](https://github.com/actualbudget/actual/commit/d158198207127830de637eb59782c4bc9018c74b) add logging - [`10ac92c`](https://github.com/actualbudget/actual/commit/10ac92c026a1be82e3ae5dbeef3b7610d292b2dd) removing old way - [`edd898b`](https://github.com/actualbudget/actual/commit/edd898beed0d8330fe73e1d07618334693a84db6) release notes - [`b82be35`](https://github.com/actualbudget/actual/commit/b82be35decd0c9fdf96b2799a4195fe0c47f6aff) adding some logs in to test mac build - [`52d6989`](https://github.com/actualbudget/actual/commit/52d6989314f4b51e613603dab80835389fac805d) repent satan - [`b4d5391`](https://github.com/actualbudget/actual/commit/b4d5391ea3f255066568658a6564795320ff2e59) Merge branch 'actualbudget:master' into electron/testing - [`21646a1`](https://github.com/actualbudget/actual/commit/21646a1863107bc6b1df085b7a0223c097a3e7d6) i caste yeee oooott sinner - [`fbbe7c1`](https://github.com/actualbudget/actual/commit/fbbe7c1d4acb8447fcc2812a8b876d21a746ca3a) derp - [`1790ea5`](https://github.com/actualbudget/actual/commit/1790ea59fef046d45705ea21fed963c6192f0db2) hmmm< ### 📊 Changes **4 files changed** (+52 additions, -14 deletions) <details> <summary>View changed files</summary> 📝 `bin/package-electron` (+2 -4) ➕ `packages/desktop-electron/beforePackHook.ts` (+36 -0) 📝 `packages/desktop-electron/package.json` (+7 -10) ➕ `upcoming-release-notes/3220.md` (+7 -0) </details> ### 📄 Description <!-- Thank you for submitting a pull request! Make sure to follow the instructions to write release notes for your PR — it should only take a minute or two: https://github.com/actualbudget/docs#writing-good-release-notes --> **Fixes:** - https://github.com/actualbudget/actual/issues/3199 - https://github.com/actualbudget/actual/issues/2705 **Root cause**: - better-sqlite3 was being built incorrectly. **Solution**: - Move away from "Universal" builds _for now_, build for x64 and arm64. - Before packing the app, detect which arch we're building for and build better-sqlite3 manually for that arch --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-02-28 21:00:33 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#4742