Review Request 111945: Use LINK_PUBLIC and LINK_PRIVATE in CMake (part 1)

Wojciech Kapuscinski wojtask9.kde at gmail.com
Fri Aug 9 12:32:59 UTC 2013



> On Aug. 9, 2013, 11:27 a.m., Kevin Ottens wrote:
> > tier1/kconfig/src/gui/CMakeLists.txt, line 15
> > <http://git.reviewboard.kde.org/r/111945/diff/1/?file=177388#file177388line15>
> >
> >     The LINK_PUBLIC is ok here, but is the Qt5::Core necessary? Doesn't it get picked up by transitivity with KConfigCore?
> >     
> >     Just wondering, I admit I'm unsure.

Qt5::Core isn't necessary. 
There was discussion on k-f-d but without clear statement what is better (add Qt5::Core or not) so I looked into all public headers and if there were classes from Qt5::Core I added to PUBLIC_LINK.

I'm not unsure if this solution is OK. Tier1 is simple but in higher tiers this could be more verbose (personally I believe code is for human, for computer it doesn't matter :) ).
Witch way I should go? :)

(NOTE: changing LINK_PUBLIC in future is BIC)


- Wojciech


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


On Aug. 8, 2013, 4:09 p.m., Wojciech Kapuscinski wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111945/
> -----------------------------------------------------------
> 
> (Updated Aug. 8, 2013, 4:09 p.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Description
> -------
> 
> Task from kdelibs cleanup.
> 
> 
> Diffs
> -----
> 
>   tier1/kconfig/src/core/CMakeLists.txt 398a1d7 
>   tier1/kconfig/autotests/kconfig_compiler/CMakeLists.txt b04b867 
>   tier1/kconfig/autotests/CMakeLists.txt 751396f 
>   tier1/kconfig/src/gui/CMakeLists.txt 91f013b 
>   tier1/kdbusaddons/autotests/CMakeLists.txt bf03734 
>   tier1/kdbusaddons/src/CMakeLists.txt 25d1f60 
>   tier1/kdbusaddons/tests/CMakeLists.txt 57e6d4f 
>   tier1/kidletime/src/CMakeLists.txt d709b2b 
>   tier1/kjs/src/kjs/CMakeLists.txt ebbb820 
>   tier1/kjs/src/kjs/api/CMakeLists.txt 4d5c5d6 
>   tier1/kplotting/autotests/CMakeLists.txt afebdee 
>   tier1/kplotting/src/CMakeLists.txt 7ce75bd 
>   tier1/kwidgetsaddons/autotests/CMakeLists.txt 870e16e 
>   tier1/kwidgetsaddons/src/CMakeLists.txt c64302a 
>   tier1/kwindowsystem/src/CMakeLists.txt fe0f920 
>   tier1/sonnet/autotests/CMakeLists.txt 88574ff 
>   tier1/sonnet/src/plugins/aspell/CMakeLists.txt 70f06d1 
>   tier1/sonnet/src/plugins/enchant/CMakeLists.txt 2319be8 
>   tier1/sonnet/src/plugins/hspell/CMakeLists.txt c7cafc5 
>   tier1/sonnet/src/plugins/hunspell/CMakeLists.txt a0e2d9b 
>   tier1/sonnet/src/ui/CMakeLists.txt 723d8f3 
>   tier1/sonnet/src/unicode/CMakeLists.txt 6b5d27d 
>   tier1/sonnet/tests/CMakeLists.txt 6e0e450 
>   tier1/threadweaver/src/WeaverGui/CMakeLists.txt f92577c 
> 
> Diff: http://git.reviewboard.kde.org/r/111945/diff/
> 
> 
> Testing
> -------
> 
> It builds and tests looks OK.
> 
> 
> Thanks,
> 
> Wojciech Kapuscinski
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130809/e044c55a/attachment.html>


More information about the Kde-frameworks-devel mailing list