Review Request 112060: Use LINK_PUBLIC and LINK_PRIVATE in CMake (part 3)

Kevin Ottens ervin at kde.org
Mon Aug 19 10:16:13 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112060/#review38124
-----------------------------------------------------------



staging/frameworkintegration/src/platformtheme/CMakeLists.txt
<http://git.reviewboard.kde.org/r/112060/#comment28192>

    The comment actually matters, please keep it. Even if that's indirect we have to link to KIO.



staging/ktextwidgets/src/CMakeLists.txt
<http://git.reviewboard.kde.org/r/112060/#comment28193>

    Useless change.



staging/ktextwidgets/src/CMakeLists.txt
<http://git.reviewboard.kde.org/r/112060/#comment28194>

    ditto


- Kevin Ottens


On Aug. 13, 2013, 5:18 p.m., Wojciech Kapuscinski wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112060/
> -----------------------------------------------------------
> 
> (Updated Aug. 13, 2013, 5:18 p.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Description
> -------
> 
> Everything in staging directory (except kde4support)
> 
> 
> Diffs
> -----
> 
>   experimental/kdeclarative/CMakeLists.txt 6c9b088 
>   staging/frameworkintegration/src/integrationplugin/CMakeLists.txt 437f0e7 
>   staging/frameworkintegration/src/platformtheme/CMakeLists.txt ec20115 
>   staging/itemviews/autotests/CMakeLists.txt a122958 
>   staging/itemviews/src/CMakeLists.txt 45b76e1 
>   staging/itemviews/tests/CMakeLists.txt 68dd537 
>   staging/kcompletion/autotests/CMakeLists.txt 25fee98 
>   staging/kcompletion/src/CMakeLists.txt 9ef378c 
>   staging/kcompletion/tests/CMakeLists.txt 22050fc 
>   staging/kconfigwidgets/autotests/CMakeLists.txt 3afeccc 
>   staging/kconfigwidgets/src/CMakeLists.txt 0ec0987 
>   staging/kconfigwidgets/tests/CMakeLists.txt 919342f 
>   staging/kcrash/src/CMakeLists.txt 5bdaf61 
>   staging/kde4attic/autotests/CMakeLists.txt 9e3d488 
>   staging/kde4attic/src/CMakeLists.txt f395cd5 
>   staging/kde4attic/tests/CMakeLists.txt fd7fd3a 
>   staging/kemoticons/autotests/CMakeLists.txt b00056a 
>   staging/kemoticons/src/core/CMakeLists.txt de6640b 
>   staging/kemoticons/src/core/kemoticonstheme.cpp 725c9c1 
>   staging/kemoticons/src/providers/adium/CMakeLists.txt 2ad9512 
>   staging/kemoticons/src/providers/kde/CMakeLists.txt 8602909 
>   staging/kemoticons/src/providers/pidgin/CMakeLists.txt 242807d 
>   staging/kemoticons/src/providers/xmpp/CMakeLists.txt 55330ce 
>   staging/kemoticons/tests/CMakeLists.txt ea7fc0b 
>   staging/kguiaddons/autotests/CMakeLists.txt 772caee 
>   staging/kguiaddons/src/CMakeLists.txt fc8941a 
>   staging/kguiaddons/tests/CMakeLists.txt ab44a37 
>   staging/ki18n/src/CMakeLists.txt f80bae6 
>   staging/kiconthemes/autotests/CMakeLists.txt 854cbe3 
>   staging/kiconthemes/src/CMakeLists.txt 150eb1a 
>   staging/kiconthemes/tests/CMakeLists.txt 49f179c 
>   staging/kinterprocesswindowing/src/CMakeLists.txt 0cb849a 
>   staging/kio/autotests/CMakeLists.txt b43ac26 
>   staging/kio/src/core/CMakeLists.txt be00bbe 
>   staging/kio/src/kssld/CMakeLists.txt f030a65 
>   staging/kio/src/widgets/CMakeLists.txt f68034c 
>   staging/kjobwidgets/autotests/CMakeLists.txt 559ba28 
>   staging/kjobwidgets/src/CMakeLists.txt e160da0 
>   staging/kjobwidgets/tests/CMakeLists.txt 7e5331a 
>   staging/knotifications/autotests/CMakeLists.txt b12bff6 
>   staging/knotifications/tests/CMakeLists.txt 91d46cf 
>   staging/kservice/autotests/CMakeLists.txt 326bbf9 
>   staging/kservice/src/CMakeLists.txt 893f6dd 
>   staging/kservice/tests/CMakeLists.txt 5e94b81 
>   staging/ktextwidgets/autotests/CMakeLists.txt 4c2b9d6 
>   staging/ktextwidgets/src/CMakeLists.txt 1f47da2 
>   staging/ktextwidgets/tests/CMakeLists.txt 3dd1376 
>   staging/kunitconversion/autotests/CMakeLists.txt c041a91 
>   staging/kunitconversion/src/kunitconversion/CMakeLists.txt 3d6771b 
>   staging/kwallet/autotests/CMakeLists.txt 95535ca 
>   staging/kwallet/src/CMakeLists.txt d181426 
>   staging/kwallet/tests/CMakeLists.txt c75ff00 
>   staging/xmlgui/autotests/CMakeLists.txt 6b415ba 
>   staging/xmlgui/src/CMakeLists.txt 9731f98 
>   staging/xmlgui/tests/CMakeLists.txt 3f17e9a 
>   staging/xmlgui/tests/krichtexteditor/CMakeLists.txt 4007776 
> 
> Diff: http://git.reviewboard.kde.org/r/112060/diff/
> 
> 
> Testing
> -------
> 
> it builds (with prefix removed) and tests looks OK.
> Testes with other changes (CMake part1, part2)
> 
> 
> Thanks,
> 
> Wojciech Kapuscinski
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130819/9f94c4a8/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list