Review Request 122498: Register with dbus to wake up KRun, and unregister then

[Christoph Cullmann]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122498/#review76057
-----------------------------------------------------------

Ship it!


I am still no sure, if that is the best way to go, but if it works around the current nasty bug, please commit and close the bug!
Thanks for taking care of the patch.

- Christoph Cullmann


On Feb. 9, 2015, 11:04 a.m., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122498/
> -----------------------------------------------------------
> 
> (Updated Feb. 9, 2015, 11:04 a.m.)
> 
> 
> Review request for Kate and KDE Frameworks.
> 
> 
> Repository: kate
> 
> 
> Description
> -------
> 
> This is the patch from bug 339094. As it seemes there's no response/activity to that, I just put it here for proper review (and will commit on behalf of Liu Zhe obviously) as this is really annoying bug. Following is the description by Liu:
> 
> Documentation of KToolInvocation (called by KRun when click a file in dolphin) said "Multi" means "Always start a new service, wait until the service has registered with D-Bus". But when the second text file is opened, kate just starts a QApplication and waits, without registering with DBus.
> In KDE 4, the second instance of kate launchs an empty KApplication to do this job, but the author said "it's too bad" to register a service. Maybe we can register to make KService happy and unregister at once.
> Another problem is that the kate icon will jump quite a few seconds after clicking the second file. I thought the reason was KStartupInfo::appStarted() not called as it requires Q_WS_X11 macro, which is not defined in Qt 5. But the bug still exists after I fixed this.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 408f502 
>   config.h.cmake d15b78e 
>   kate/src/kateapp.cpp 4c26f33 
>   kate/src/main.cpp 8852754 
> 
> Diff: https://git.reviewboard.kde.org/r/122498/diff/
> 
> 
> Testing
> -------
> 
> Hrvoje confirmed in the bug report that this indeed fixes the issue.
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150215/17f637c8/attachment.html>