-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[Bug]: Desktop app on macOS budget loading error #2705
Comments
I'm having the same issue. |
Can someone try out the build available here and confirm if its working for you or not? https://github.com/actualbudget/actual/actions/runs/8971187425?pr=2715 It should fix the loading issue. You will need to download the file listed as "actual-electron-macos-latest". That is a zip file that contains the desktop app. |
That fixed it for me! |
Excellent! Thanks for the updated file. Works for me. |
This seems to be broken again. |
Ok, so I'm really confused. Building the desktop app locally works just fine. Building the same source via CI/CD - creates a broken build. Code signing and other things are all turned off. So I'm really lost as to what is going on which makes it very hard to debug and patch the issue. |
@MatissJanis Could you upload a good working build to the release? That way people can use it while we try to determine why the autobuild is broken. |
Good idea. Uploaded my local build to the release as a temporary solution: https://github.com/actualbudget/actual/releases/tag/v24.6.0 |
@youngcw had a brilliant idea: the issue might have to do with the Github CI runners. Here's a CI job that was building a working artefact: https://github.com/actualbudget/actual/actions/runs/8971187425/attempts/1 So I re-ran the same exact job and it failed: https://github.com/actualbudget/actual/actions/runs/8971187425/attempts/2 This points to the issue being in the runner. Now we just need to figure out what changed and how to change it back. |
@MatissJanis I would guess the issue is the non-pinned OS version: The passing run:
and the failing run:
What exactly changed in macOS that is causing the issue, I do not know. But other than that, and the fact the artifact size is slightly different, I do not spot any differences in the logs of the two runs. |
Any ideas how to pin to a specific runner version? |
Looks like we can only pin to major versions so even if we specified macos-14 we would probably still have the same issue |
Maybe this project could help. https://github.com/pd4d10/debugtron |
AFAIK you can not. You can only select major release, e.g. |
I used the debugtron and see this when opening a local file:
|
Have we ever confirmed if the mac images fail on both Intel and M1/M2 macs? I am on an Intel mac. GitHub's latest macos image is arm only. |
I'm on Intel too. |
I'm on Apple silicon and the original released 24.6.0 image fails for me, suggesting it is not an architecture specific issue. |
Maybe related: actions/runner-images#9995 |
Between 24.5.0 and 24.6.0 better-sqlite3 was upgraded from 9.1 to 9.6, between those versions better-sqlite3 was updated to support native arm builds on macOS, perhaps something broke there if the GitHub runner changed from x86 to arm between working and not working? |
This is quite the rabbit hole. I'm wondering if the actions/cache is caching the node_modules from the previous arm node runtime. Something like this: |
It looks like it is- the cache key is only based on the os, not the architecture. So the now x86/x64 node runtime is using the cached node_modules for arm, which is likely where the "Bindings not found." error is coming from during the Electron build step. |
I'm encountering this on the most recently released version:
|
I'm experiencing this bug on the latest version also. |
To keep this thread in the loop, I think it's fixed here: https://github.com/actualbudget/actual/actions/runs/10322929844?pr=3220 For those with Arm CPU's, can you try out the latest build and tell me what you see? I think some of you may see "this file is damaged" due to some Apple nonsense about notarizing the app (we plan on fixing that). If you see the damaged file warning, I think running the below command will fix it:
Can someone confirm? |
Hello @MikesGlitch , I just downloaded and installed the current Release today [v24.8.0]. I still have this error. However, as advised by @Kidglove57 above, I enabled Rosetta and that resolved the issue for now. Machine: |
Thanks for checking. I think I fixed it here, can I get someone to try again? https://github.com/actualbudget/actual/actions/runs/10339466938?pr=3220 Apologies for all this back and forward, I don't have a Arm64 mac so can't do it myself. |
✅ This works on Intel if that helps (don't have an ARM machine). |
I have a PR up with the fix, thanks for the help everyone! 🚀 |
Fixed by #3220 |
It worked for me too! (M1Pro) Thanks! |
Sadly still present for me :( Also M1 pro MBP 14" |
Can you confirm that you have the latest artefact and have run the command? If so, what error do you get? |
I'm testing the latest build on my M2 Mac. As expected I get the damaged file message. I tried running the command you mentioned. However Terminal is throwing an error: |
Are you running the command with the <> brackets? If you remove them it should work A real life command might look similar to
|
The same as before, with an "unknown error opening x file". My server is running the latest version. I had to run the xattr command to get it to work |
This is odd because others have reported it working - and an arm64 should work the same no matter what machine its on. Can you try removing it from your system, then re-downloading it from https://github.com/actualbudget/actual/releases/tag/v24.8.0 ? If you still get the error can you open up devtools by clicking the "View" menu > "toggle devtools" > "Console" until you get the error and screen grab it for me? |
Still struggling a bit in terminal so? |
For me it was:
|
Thank you for persevering with me. I'm not sure what I was doing wrong - maybe incorrect spacing in the command. Anyhow, now working really well and I can confirm that export works too. Just in case others are reading this - don't expect any response when you enter this command in terminal. But it does work |
Here is the error: The previous release still works fine. Edit: actually I see I'm using the intel release (unbeknownst to me) for the previous version - do I need to match the server architecture and my laptop architecture? |
Can we take this to Discord? Hard to have a convo on Github. Here's the link to the support thread: https://discord.com/channels/937901803608096828/1271969489294200883 |
Hello @MikesGlitch, not sure if it related to this issue, but the export was error in macOS. |
That error was unrelated, but it's fixed in the latest release as well (you'll have to download it again to get it). We also have automatic backups in the Electron app - there's not much reason to do a manual export 👍 |
Verified issue does not already exist?
What happened?
I successfully downloaded the desktop app on my Mac and when I connect to my fly.io account and load the budget I’m getting an error. Errors are “We had an unknown problem opening “My Finance dcb1c51” and I’m also unable to browse to the exported .json file. Occasionally I’m also getting the “something went wrong trying to download that file error”.
Additional information:
I’m facing this issue on- Client version: v24.5.0
Server version: v24.5.0
Screenshots attached.
Where are you hosting Actual?
Fly.io
What browsers are you seeing the problem on?
Safari
Operating System
Mac OSX
The text was updated successfully, but these errors were encountered: