Review Request 123907: Make log viewer usable in multiple instances + fix the "KDEInit couldn't launch log viewer" error and Plasma being blocked
Martin Klapetek
martin.klapetek at gmail.com
Tue May 26 13:30:26 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123907/
-----------------------------------------------------------
Review request for Release Team and Telepathy.
Bugs: 346395
http://bugs.kde.org/show_bug.cgi?id=346395
Repository: ktp-text-ui
Description
-------
This fixes bug 346395 - when it is launched from Plasma, it will block all of Plasma until the app quits, which is announced by an error message box with that "KDEInit couldn't launch" error.
I'd like to commit this to stable too however this adds a new dependency on KDBusAddons. I think the severity of this issue warrants that new dependency, but can someone from the release team please ack this?
Additionally it makes it "Multiple" allowing for multiple running instances. I think it can be useful to eg. compare logs, but if anyone has a good reason for why it shouldn't be Multiple, please speak up.
Diffs
-----
CMakeLists.txt 72c2f8f
logviewer/CMakeLists.txt ac35ff6
logviewer/ktp-log-viewer.desktop 1674566
logviewer/main.cpp 00dbc03
logviewer/org.kde.ktplogviewer.desktop PRE-CREATION
Diff: https://git.reviewboard.kde.org/r/123907/diff/
Testing
-------
Plasma is no longer blocked when the log viewer is launched from Kicker.
Thanks,
Martin Klapetek
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/release-team/attachments/20150526/fe4879b6/attachment.html>
More information about the release-team
mailing list