plasma-applet-thermal-monitor icon indicating copy to clipboard operation
plasma-applet-thermal-monitor copied to clipboard

Published 20 hours ago • verified publisher icon kotelnik

Project status?

Open shmerl opened this issue 5 years ago • 10 comments

There are a bunch of issues and pull requests with fixes that are sitting around for a while. @kotelnik: are you still maintaining the project?

shmerl avatar Feb 24 '20 04:02 shmerl

this project is dead, he is no more active over 1.5 years

alium avatar Jun 10 '20 21:06 alium

Is there any active fork? The author didn't respond in a long time, so he isn't available.

shmerl avatar Jun 10 '20 22:06 shmerl

Or is there a good alternative? Everything I have tried so far is buggy and crash prone.

patlefort avatar Jul 25 '20 04:07 patlefort

I did not find any alternative

use this two patch, with this patchs works wigdet again for me under Qt 5.15: https://aur.archlinux.org/cgit/aur.git/tree/0001-let-var-systemmonitorAvailableSources-refer-to-an-empty-array.patch?h=plasma5-applets-thermal-monitor-git https://aur.archlinux.org/cgit/aur.git/tree/0002-Fix-Plasmashell-crash-when-opening-config.patch?h=plasma5-applets-thermal-monitor-git

alium avatar Jul 25 '20 07:07 alium

Is there any active fork? The author didn't respond in a long time, so he isn't available.

I'd be happy to fork it and maintain it. I don't really know JavaScript that well, but I would be happy to at least review and test PRs.

zhimsel avatar Jul 26 '20 14:07 zhimsel

Is there any active fork? The author didn't respond in a long time, so he isn't available.

I'd be happy to fork it and maintain it. I don't really know JavaScript that well, but I would be happy to at least review and test PRs.

just do it ;-)

alium avatar Jul 28 '20 08:07 alium

I've actually forked it here quite some time ago exactly to merge some patches and not do it manually. It's a gitlab instead of github though but I'm happy to merge and review PRs although my experience is also limited. Also happy to grant permissions for others, so it's not a one man show. Let me know if you're interested

agurenko avatar Jul 28 '20 08:07 agurenko

nice @agurenko . i asked for switching to your fork https://bugs.archlinux.org/task/67375

alium avatar Jul 30 '20 07:07 alium

@agurenko I've tested your fork on my system here, and I'm still getting that crash when opening the config window. Using the upstream repo with the patches listed above (see these two patches), the crash does not occur. I can dig into this more when I have some free time, but otherwise, would you mind investigating and merging that patch in with your fork? I'd be happy to switch the AUR package to use it.

zhimsel avatar Jul 31 '20 05:07 zhimsel

@agurenko Okay, I've managed to merge in those changes to your fork. I'll be creating a PR for you shortly. Once that's merged, I'll switch the AUR package over. I'll also create a BIG BOLD LETTERS issue in this repo to make it easier for people to find your fork when browsing this repo.

EDIT: https://gitlab.com/agurenko/plasma-applet-thermal-monitor/-/merge_requests/4

zhimsel avatar Jul 31 '20 15:07 zhimsel