Review Request 116769: use TelepathyQt4Farstream pkg-config flags

[David Edmundson]

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



libqtf/CMakeLists.txt
<https://git.reviewboard.kde.org/r/116769/#comment37209>

    What's the difference between
    
        ${TP_QT4_FARSTREAM_INCLUDE_DIRS}
    and
        ${TELEPATHY_QT4_FARSTREAM_INCLUDE_DIR}
    
    It looks like one of these must be redundant?
    


- David Edmundson


On March 12, 2014, 3:53 p.m., Rex Dieter wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/116769/
> -----------------------------------------------------------
> 
> (Updated March 12, 2014, 3:53 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Repository: ktp-call-ui
> 
> 
> Description
> -------
> 
> Use TelepathyQt4Farstream pkg-config flags instead of telepathy-farstream.  In a perfect world these match,
> but when/if one has to install telepathy-farstream-0.4 in parallel and/or in a different place than telepathy-qt, things go wrong.
> 
> This is one small step in fixing
> BUG: 312450
> 
> 
> Diffs
> -----
> 
>   libqtf/CMakeLists.txt 38d234b 
> 
> Diff: https://git.reviewboard.kde.org/r/116769/diff/
> 
> 
> Testing
> -------
> 
> Been using this to build ktp-call-ui packages in fedora for quite awhile, due to having to install telepathy-farstream-0.4 in parallel (and in a different header prefix) to newer gst1-based telepathy-farstream-0.6
> 
> 
> Thanks,
> 
> Rex Dieter
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20140312/96875c3f/attachment-0001.html>