-
Notifications
You must be signed in to change notification settings - Fork 13
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
Brewlet 1.7 broken under macOS Ventura 13.2 #34
Comments
Thanks for bringing this up! I'll look into it and hopefully resolve it by the weekend. |
Hey, I upgraded to macos 13.2 and haven't run into any issues yet, so I need some more information:
Let me know if you need more info about the first step. Thanks |
Hi, i´ve the same problem since 13.2 so I try to answer the questions
|
This is good info, thanks. I see a clue in the log:
Setting a menu item title is failing for some reason, more than likely due to the name or version of a package (e.g., maybe the string is too long, or is in a different format or encoding). I assume Brewlet only crashes if there formulae that need upgrading. It would also help to know what formulae you have on your system, so please attach a file of the output of |
Also, if you're able to reproduce the error, can you do the following:
This can help confirm where the error is coming up from. Because I'm not completely convinced it's due to the |
Sorry, didn´t know why, but brewlet doesn´t crash anymore. |
Ah I see. I think the unexpected error comes from parsing the JSON output, so the above debugging steps can still help me diagnose/fix it |
Didn´t know - do You need some more Infos? |
Yea that would be great. Both streaming to find the exact error and the output of |
Hi, today Brewlet crashs during the streaming. Console-Stream: |
Ok, I have ideas about what may be causing this from this line in the crash:
From here it seems like Brewlet is reaching a limit to how many threads can be open on the main dispatch queue. There shouldn't be many reasons why this happens but my guess is that there are casks/formulae that require user interaction, thus when they are in the background, they will hang. As Brewlet runs, it can request more upgrade commands and they will hang, and fill up the queue. Can you verify the following things?
In the meantime I will merge the fix for #31 so we can test without casks and see if the problem persists. Thanks for your patience as this is very much a side project for me and macos/Swift is definitely not my expertise. (As an aside, I don't know why this error would result in a segfault as in the logs, |
Auto-upgrade was set off |
Brewlet 1.7 seems to be working after upgrading to macOS Ventura 13.2.1 and updating Homebrew! |
Yes since 13.2.1 it works again |
and after 5 days the problem is back again using 13.2.1 :-( |
Ah :( I’ll look into if they are making changes to the JSON API that is causing it to fail
|
It seems there were some updates to the JSON command, but none that I can see that would cause it to break, then be okay, then break again. Since Brewlet is working for some (myself included), I expect it to come to individual differences. I would love to replicate the issue so I can solve it, and it would help if I have the output of |
Hope the Output will help - thx for watching the problem! (it really seems to have something to do with the programs waiting to be updated. Just now brewlet works normally again. When it didn't work for me, there were 6 updates pending, which I did via the terminal. Unfortunately I don't know which ones they were) |
I tried a couple of things, with no success unfortunately:
I added more detailed error messages to the notifications I send so we can zero-in on where the error is coming from. You can download the zip from the prerelease here and test it out. I suspect if it's hard crashing then this won't help because it won't get a chance to output the error. If it's truly in the DispatchQueue, we can try commenting out the UI updates and see if that fixes the problem. If it does, then we can be sure where the problem is, but don't know why the thread is overcommitting. And why it only happens for some users.. |
Thx for yur work! |
Oops it was just a draft release, now I published it. Maybe it is because of a specific update. Will keep an eye on this! |
For me it suddenly worked again today. I installed 1.7.1 yesterday or two days ago. |
Same behavior on my machine |
I see, another instance of this error |
I just tried installing the latest pre-release version. The result is the same. Perhaps it was not recompiled in the archive I downloaded, I attached a link from where I downloaded it. https://github.com/zkokaja/Brewlet/releases/tag/v1.7.2#:~:text=3-,Brewlet.zip,-3.36%20MB I'm also attaching the log, maybe it will help to diagnose the problem. |
ulimit -a-t: cpu time (seconds) unlimited In my case, the application cannot start at all. For some time, the icon in the menu bar is shown, then an error occurs and the application crashes. During loading, the application is unable to interact with any elements. |
Could you upload the latest version to Homebrew Formulae? brewlet itself instructs as below:
|
The 1.7.* versions are for testing the specific bug in this issue. However, I will revert those debugging changes and create a new version soon. If anyone is experiencing this issue consistently, I urge you to clone this repo, and run the application from XCode so we can track find where it is breaking exactly. |
Good morning, I haven't had any problems with the test version since then, but the icons are no longer displayed correctly for me. It feels like a random icon is taken. Since I use bartenders, that's not quite as ideal :-) |
I found Homebrew Formulae were updated to Brewlet 1.7.4 and haven't have any problem. |
Brewlet 1.7 stuck in checking... and updating....
Getting Unexpected Error: "An unexpected error occurred. See Log for details."
The text was updated successfully, but these errors were encountered: