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