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

(Updated May 28, 2015, 10:03 p.m.)


Status
------

This change has been marked as submitted.


Review request for Release Team and Telepathy.


Changes
-------

Submitted with commit 50a901a06733299af2238fc4c4d26d2f8052dae0 by Martin 
Klapetek to branch Applications/15.04.


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

_______________________________________________
KDE-Telepathy mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kde-telepathy

Reply via email to