Review Request 120795: [ktp-auth-handler] Make it build

Hrvoje Senjan hrvoje.senjan at gmail.com
Sat Oct 25 12:18:11 UTC 2014



> On Oct. 25, 2014, 12:14 p.m., Martin Klapetek wrote:
> > CMakeLists.txt, line 67
> > <https://git.reviewboard.kde.org/r/120795/diff/1/?file=322019#file322019line67>
> >
> >     That doesn't sound right; especially since qca can have random suffixes; this should always use the CMake variable

i used this, cause it's exported target name


- Hrvoje


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


On Oct. 25, 2014, 7:21 a.m., Hrvoje Senjan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120795/
> -----------------------------------------------------------
> 
> (Updated Oct. 25, 2014, 7:21 a.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Repository: ktp-auth-handler
> 
> 
> Description
> -------
> 
> plus other misc. things.. can split the patches up, as not everything's related
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 5994416 
>   cmake/modules/FindKTp.cmake 8facd12 
>   cmake/modules/FindQJSON.cmake 4781c0a 
>   main.cpp 19f4b07 
>   org.freedesktop.Telepathy.Client.KTp.ConfAuthObserver.service.in 928a71d 
>   org.freedesktop.Telepathy.Client.KTp.SASLHandler.service.in b29d7e1 
>   org.freedesktop.Telepathy.Client.KTp.TLSHandler.service.in e8d6f0f 
> 
> Diff: https://git.reviewboard.kde.org/r/120795/diff/
> 
> 
> Testing
> -------
> 
> builds against latest frameworks ktp-c-i
> 
> 
> Thanks,
> 
> Hrvoje Senjan
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20141025/6e9e9792/attachment.html>


More information about the KDE-Telepathy mailing list