Review Request 111683: Make TelepathyLoggerQt an optional dependency of ktp-text-ui
Dan Vrátil
dvratil at redhat.com
Thu Jul 25 14:43:42 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111683/
-----------------------------------------------------------
Review request for Telepathy.
Description
-------
Because LogManager is asynchronous, #ifdef'ing it's usage would be very messy, so instead I introduced a dummy implementation which only emits fetched() signal with empty list when building without TpLoggerQt. Given it's just a temporary measure, this seems to be the least intrusive approach.
Diffs
-----
CMakeLists.txt 1d16dc7
lib/CMakeLists.txt 80a3eff
lib/chat-widget.cpp 74cee48
lib/logmanager.h 3f4a2f1
lib/logmanager.cpp 6cd1aec
lib/logmanager_dummy.cpp PRE-CREATION
Diff: http://git.reviewboard.kde.org/r/111683/diff/
Testing
-------
Thanks,
Dan Vrátil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20130725/cd8ab76d/attachment.html>
More information about the KDE-Telepathy
mailing list