-
Notifications
You must be signed in to change notification settings - Fork 7
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
please update for Plasma 6 #12
Comments
Thanks, and regarding your question: yes it can, but it may take some time. Firstly, porting to Plasma 6 seems not to be trivial. Secondly, Workrave itself crashes on KDE's Wayland, and the version that probably doesn't crash is still in beta. |
I've updated my plasmoid to Plasma 6. Here is how you can try it out:
I still haven't switched to Plasma 6 so it's mainly untested. The applet should work in the X11 session. Stable Workrave crashes constantly on Wayland. I didn't test the beta version that says it has fixes for Wayland. |
you would like me to test or might as well wait for the Workrave update? I can test if you are requesting it |
You asked for a Plasma 6 version of the plasmoid. It's here for you to use it. If plain Workrave works for you, then my plasmoid should work as well. Not requesting anything. |
I just meant about testing sooner vs later. I very much want to use this, but Workrave itself is not yet working for me, so the full functioning will be both Workrave and this plasmoid. And if you would like me to test the plasmoid even though Workrave is still needing update to support kde-idle, I could do that. Otherwise, I will wait for Workrave to be updated and then test this then. Thanks so much! |
Testing my plasmoid while Workrave is not working won't help me. Thanks anyway. |
okay, when Workrave is updated to work with KDE 6, I'll happily then use it with your plasmoid |
@wolftune How do you compile the beta Workrave? I know that to get it to work under LabWC, I had to add |
@jjramsey I would ask that at https://github.com/rcaelers/workrave/discussions not on this issue about the applet |
Just FYI, I wasn't asking for advice on compiling Workrave. I was asking how you compile it, because I wanted to know if you had done it correctly. You had commented on the issue about Workrave's Wayland support (rcaelers/workrave#523), so I figured you had at least tried the Workrave beta.
FWIW, I had been keeping my eye on the idle-notify protocol, and as far as I can tell, it started as a KWin protocol and then got adapted into a proper Wayland one (see the next to last paragraph in https://www.phoronix.com/news/Wayland-Protocols-1.27). It should be implemented in KWin now (https://invent.kde.org/plasma/kwin/-/merge_requests/2959). |
I can verify that the Plasma 6 version of the applet works for me, using the Workrave beta compiled for Wayland. |
I want to note that on OpenSUSE Tumbleweed the qdbus executable is called
I believe the solution is to add this to Plasma5Support.DataSource {
id: qDBusQt6AlternativeNameCheck
engine: 'executable'
interval: 0
connectedSources: ["qdbus6"]
onNewData: function(sourceName, data) {
if (!interval)
return
if (data.stderr.length > 0) {
} else {
dbusExecutableName = "qdbus6"
}
workraveCheckDBus.interval = 1
interval = 0
}
} |
@arturaz, thank you for this valuable piece of information. I'll be checking for qdbus6 as well. |
After using it for some time I can confirm that Plasma 6 version works perfectly fine on X11 session as well.
|
qdbus-qt6 is needed on Fedora, so I cannot remove it. I've added a script to the applet to detect the qdbus name used by user's distribution. Here is how you can try it out:
if you already have installed my applet, or
if you don't have it installed at the moment. |
Hi, I gave up on waiting for the full Wayland release and had other wayland problems, so I tried switching my system back to x. Workrave is working (not the beta, the normal older version), but the widget is not. I'm seeing the qdbus error message. I'm using KDE Neon. |
This one?
Does this command work for you? |
Yes, that message, and it works as in:
I don't have |
My bad. I wrote a script for detecting a qdbus name that doesn't work on Ubuntu's dash. I just pushed a fix to the kf6-port branch. You can update your applet as explained in this comment or copy this file to your Thanks a lot for reporting this. |
Works!! Yay! Guessing I don't need |
Glad to hear 🙂 I'm not sure about |
qt6-tools-dev not needed indeed, confirming I think the only thing left to close this issue is to do a normal release |
The normal release has been done as the 0.4.0 version, so closing now. |
FYI, I see the README updated, but the GitHub "about" section still says "KDE Plasma 5 applet for displaying Workrave timers." |
Indeed 🙂 Thanks! |
I love this, but it won't work in plasma 6. Can it get updated?
The text was updated successfully, but these errors were encountered: