Bug 350979 - kteatime freezes plasma when run from application launcher
Summary: kteatime freezes plasma when run from application launcher
Status: RESOLVED FIXED
Alias: None
Product: kteatime
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Ubuntu Linux
: NOR major
Target Milestone: ---
Assignee: Stefan Böhmann
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-08-04 22:53 UTC by James Cain
Modified: 2015-12-05 17:05 UTC (History)
7 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Proposed fix for freeze when started from krunner (636 bytes, patch)
2015-08-28 17:04 UTC, Alexander Kowalski
Details

Note You need to log in before you can comment on or make changes to this bug.
Description James Cain 2015-08-04 22:53:24 UTC
KDE 5.3.0
KDE Frameworks 5.9
Netrunner 16  (Kubuntu 15.04)
kteatime version 1.3

When run from terminal kteatime runs fine. If launched from the KDE application launcher the Plasma Desktop hard-freezes, requiring a system re-boot from terminal. When run from terminal, no errors are given.
Comment 1 Alexander Kowalski 2015-08-15 13:34:21 UTC
For me, krunner would freeze from starting kteatime until closing it again, after which it would show the error message "KDEInit could not launch '/usr/bin/kteatime'". (With Plasma 5.3.95 and Applications 15.07.90)

You should try deleting the line "X-DBUS-StartupType=Multi" (or putting None instead of Multi) at the end of the file /usr/share/applications/org.kde.kteatime.desktop. If your problem is the same as mine, that should fix it.

Also, it seems to me (from looking into it quickly) that kteatime doesn't register with D-Bus directly anyway (only saw something for the status notifier item), so this should actually probably not be there.
Comment 2 HT 2015-08-28 15:56:30 UTC
Hello,

same problem, kteatime freezes when is runnig from krunner. Krunner and plasma freeze too. Starting application from terminal works fine. 

I have seen this bug in KDE4, now I'm runnig plasma 5.4.0 and bug is stil present. Tested with clean profile too.

Bug reproducible: Always

Steps to reproduce:
1. Start KDE
2. Run krunner 
3. Search and run kteatime
Comment 3 Alexander Kowalski 2015-08-28 17:04:46 UTC
Created attachment 94263 [details]
Proposed fix for freeze when started from krunner

What I described in my first comment should fix the problem with krunner, but I had hoped to get a reply whether this also fixes the originally reported bug, because I am not really sure if it actually is the same. I could always start kteatime without problems from the Application Launcher (if that is supposed to mean the menu plasmoid of that name) and plasma would also not be affected when starting from krunner.

Since that was quite some time ago, I also attach the patch corresponding to my comment, but it would be good if someone with the exact problem described in the bug report would check if that is also fixed.
Comment 4 James Cain 2015-08-28 17:49:12 UTC
(In reply to Alexander Kowalski from comment #3)

> What I described in my first comment should fix the problem with krunner,
> but I had hoped to get a reply whether this also fixes the originally
> reported bug, because I am not really sure if it actually is the same.

Your proposed fix does solve the problem :)

> I could always start kteatime without problems from the Application Launcher
> (if that is supposed to mean the menu plasmoid of that name) 

I did indeed mean the application launcher of the same name. 

> Since that was quite some time ago, I also attach the patch corresponding to
> my comment, but it would be good if someone with the exact problem described
> in the bug report would check if that is also fixed.

It does indeed appear that the patch fixes the problem, for me at least. Thank you.

- James
Comment 5 Christoph Feck 2015-09-27 20:51:08 UTC
Alexander, thanks for the patch. If possible, please post it to https://git.reviewboard.kde.org/
Comment 6 Ovidiu-Florin BOGDAN 2015-10-05 07:03:09 UTC
I had the same problem with Kompare. The proposed patch fixed it.
Comment 7 Luigi Toscano 2015-12-01 23:47:25 UTC
Adding Andrius, who did the port to Frameworks 5. It seems indeed that kteatime does not expose a dbus interface anymore.
Comment 8 Andrius Štikonas 2015-12-05 16:56:01 UTC
Ok, I committed the patch in http://commits.kde.org/kteatime/d2922c0b8fcf4f500908d826073907ed26453ec7

Thanks for reporting.
Comment 9 Luigi Toscano 2015-12-05 17:00:51 UTC
(In reply to Andrius Štikonas from comment #8)
> Ok, I committed the patch in
> http://commits.kde.org/kteatime/d2922c0b8fcf4f500908d826073907ed26453ec7
> 
> Thanks for reporting.

So the removal of dbus interface in the Frameworks version is a planned choice (not questioning, just for reference; the alternative to this problem would have been reenabling the dbus interface).
(PS: I think you know and you just forgot, but if you use BUG: nnnnn in a separate line in the commit message, the bug is closed automatically).
Comment 10 Andrius Štikonas 2015-12-05 17:05:46 UTC
Well, dbus interface can be reenabled. But it's probably harder and less safe to do just before the release of 15.12. I also don't know much of dbus myself :(...