-
Notifications
You must be signed in to change notification settings - Fork 52
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
Plasma 6 Support #66
Comments
Used it for 7 years and would very much appreciate a Plasma 6 compatibility. |
Same here |
I've done a quick-and-dirty port to Plasma 6 in my fork: https://github.com/Iridias/plasma-simpleMonitor Maybe I'll open a PR - but my last PR with bugfixes for proper Plasma 5.27 support is still open to this day - so I guess there is no need to hurry. |
Thanks for that! I tried to compile and install, which looked good so far: Update the VERSION argument value or use a ... suffix to tell -- Found X11: /usr/include Update the VERSION argument value or use a ... suffix to tell CMake Warning at /usr/lib64/cmake/KF5Package/KF5PackageMacros.cmake:46 (message): CMake Warning at /usr/lib64/cmake/KF5CoreAddons/KF5CoreAddonsMacros.cmake:37 (message): -- Configuring done (0.5s) docb@X1E:~/buildservice/SimpleMonitor/plasma-simpleMonitor/build> sudo make install However, when I want to install it it does not show up in the list of apps. I'm using openSUSE Tumbleweed |
You sure, you're on the right branch? The changes are in "main" not "master". |
Well, I followed the instructions... |
Checkout with --branch did not work, downloaded source manually, compiled, and result looks good! Thank you for your update! |
I downloaded the source and used cmake. As far as I can tell everything worked fine. When I tried to put the simple monitor to my desktop, plasma complained about incompatibility with V6. It seems the plasma shell considers it a V5 plasmoid. I am on Arch and Arch does not provide a plasma-framework-devel package. On Arch I only find a plasma-framework5, which is installed on my system. Any advice? |
Hi Iridias, and first, thanks for that port :) I'm happy to get simple monitor back again :) Running under plasma 6.0.4 on Opensuse Tumbleweed, I've followed the "plasmapkg2 -i plasma-simpleMonitor-0.6.plasmoid" way and had no problem with the installation. The first problem I got running the widget was an error in the widget about QtQuick.Controls version 1 missing. I've solved it by forcing the import QtQuick.controls to an upper version in /contents/ui/main.qml (works with these : 2.12 or 6.0). I can put a PR on your repo if you like. Now the widgets works but has a few issues : the misc. tab in config menu doesn't work, and CPU temperatures only show for the first 4 cores on mu 8-core system. That might be linked, because as far as I remember, the CPU config is precisely in the misc screen ;) Thanks again ! |
That might also be due to the way, the temperatures are fetched now - using the "brute force" method by parsing sensors ...not quite elegant, but at least it works
right. I fixed that in the last commit |
Plasma 6 is out any chance of this getting ported?
The text was updated successfully, but these errors were encountered: