Review Request 121539: Remove the name from TelepathyHandlerApplication

Martin Klapetek martin.klapetek at gmail.com
Mon Dec 15 18:59:20 UTC 2014


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

Review request for Telepathy.


Repository: ktp-common-internals


Description
-------

This makes all ktp parts using TelepathyHandlerApplication use the name "KDE Telepathy", which may be good but it's causing other problems like text-ui not finding its xml .rc file because it looks in .../KDE Telepathy/... folder rather than .../ktp-text-ui/... where it is installed. Alternative solution is to make everything install into KDE Telepathy subfolders.

I'm not sure which approach is better (hence this review), here's removing the default and having each ktp part set its own app name.


Diffs
-----

  KTp/telepathy-handler-application.cpp 8ebf537 

Diff: https://git.reviewboard.kde.org/r/121539/diff/


Testing
-------

ktp-text-ui gets full toolbar again


Thanks,

Martin Klapetek

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20141215/3d449177/attachment.html>


More information about the KDE-Telepathy mailing list