exequtic / apdatifier

KDE Plasma widget that tracks updates for ArchLinux, Plasma Widgets, and Flatpak, providing notifications for new ones
https://store.kde.org/p/2135796
MIT License
87 stars 7 forks source link

Widget causing Plasmashell to crash sometimes #34

Open DarkXero-dev opened 2 months ago

DarkXero-dev commented 2 months ago

Hey my friend

I dunno why, but when widget initiates an update check sometimes plasmashell decides to crash on both X11 and Wayland. I sent report to KDE devs they just said not related to Plasma. I am unsure if it's actually your widget causing it or not.. If not disregard.

I don't have any other widgets running only Apdatifier.

exequtic commented 2 months ago

Hi. Need to find out if this widget is really causing the crash. Try disabling the widget or turning off the auto-check interval to see if the crashes persist. Were there any crashes before, or did they start only after the latest Plasma update?

DarkXero-dev commented 2 months ago

Thanks for replying. @exequtic

To be honest I have had auto-check disabled for past few days. Still crashes persist. I forgot about that. Crashes were happening much less before on Plasma 6.0.x now it's becoming too annoying. Borderline unusable. As I said I reported to KDE devs they weren't helpful at all.

I am not blaming your widget but since it's the only active one I wondered. If that helps here's my report >> Here

It's frustrating not knowing what is causing this crash.

Shehata-git commented 2 months ago

Can you check the "Journald Browser" or "KSystemLog" provided in kde? Maybe you can get a bit more context on what happened beforehand for a more helpful outcome.

QuarkZ26 commented 1 month ago

I have to say, I feel like there's definitely a correlation. Twice now, I have observed with my own eye Plasma crashing/restarting just as Apdatifier started an update check.

image

You can see here that it crashes just about once a day. I've since then paused Apdatifier and only use manual check and there hasn't been a crash since the 16th.

Now I can't say whether it's a Plasma problem or the widget, I really only use this one (that isn't a default Plasma one), but I do think something is tied to this.

DarkXero-dev commented 1 month ago

There you go. Sorry was very busy with my projects and potential war here. But yes I can confirm the behavior.

It happened on my laptop during Kernel update and rebuild which in turn resulted in unbootable system that I fixed via chroot.

Issue has been there since Plasma 6.1.x so no clue.

QuarkZ26 commented 1 month ago

To be clear, it never happened to me during an update, the only times I've been able to see this is when the widget is setup to auto check and starts an actual check, as in, it starts and Plasma immediately crashes and restarts.

It never really created any issues for me beyond that, but seeing it happens twice as soon as the auto check started made me suspicious, although I'm not sure how I could confirm the times I'm not in front of the machine since I don't believe there's any log for the widget? Something that would show each time the auto check happens.

DarkXero-dev commented 1 month ago

For me it almost always happens an update initiated through widget. It's really annoying. I can't live without it so I put up with it.

Requires a lot of investigation which I have no time for right now coz a war is about to blow here gotta find shelter soon.

exequtic commented 1 day ago

I still can’t reproduce the crash when checking for updates, which makes it difficult to troubleshoot blindly. However, I was able to reproduce a crash during a system upgrade, and that led me to make some experimental changes. With these changes, the crash no longer occurs during upgrades. It’s possible that they might also prevent crashes during update checks for you. Please test the latest commit and let me know the results.

git clone https://github.com/exequtic/apdatifier.git
kpackagetool6 -t Plasma/Applet -u apdatifier/package

Make sure to restart plasmashell or Log Out after this.

DarkXero-dev commented 1 day ago

No crashes anymore but wrapper is broke

wrapper

also Terminal

term

exequtic commented 1 day ago

I'll fix this a bit later. It happens because the file $HOME/.config/apdatifier/news.json is missing. Check the news or manually create the file with empty [] inside and restart plasmashell.

DarkXero-dev commented 1 day ago

I just did and now it works

Question, since I installed from git will it still get updated via widget itself or not ?

exequtic commented 1 day ago

since I installed from git will it still get updated via widget itself or not ?

Yep, just like usual.

DarkXero-dev commented 1 day ago

@exequtic

Excellent, will update the XeroLinux rice soon.. Maybe bump version to 2.9.0 ?

exequtic commented 1 day ago

I’m always worried about releasing a new version - something always ends up breaking that I didn’t catch. I’ve made significant under-the-hood changes since the last version, so I need to make sure everything works properly. Most likely, I’ll release 2.9 in the middle of next week.

DarkXero-dev commented 1 day ago

@exequtic

I agree, better safe than sorry. It's always a good precaution. Well, will wait until you do. No sense in pushing git version to the masses as you so well pointed out ;)