Review Request 123031: Let Kross be useable w/o searching for private deps

[Alex Merry]

> On June 14, 2015, 10:36 p.m., Alexander Potashev wrote:
> > Please add "find_dependency(Qt5Core @REQUIRED_QT_VERSION@)" for completeness.
> > 
> > Build of KrossUi currently cannot be disabled. If we wanted to aim server environments without QtWidgets, then we first need to make KrossUi optional or move it into a separate framework. Until we are done with making KrossUi optional, there is almost no point in dropping find_dependency(Qt5Widgets [...]).
> 
> Hrvoje Senjan wrote:
>     >Please add "find_dependency(Qt5Core @REQUIRED_QT_VERSION@)" for completeness.
>     
>     I disagree ;-)
>     Qt5Core is nowhere to be found in link libraries (offcourse it's used, but if we want to be literal...).
>     I can add it though if we agree to also add the Qt5Core target explicitly in link_libraries.

Yeah, my view is that you should be searching for anything that is in the list of interface link libraries for exported targets, but not for anything else. If you drag it in implicitly via `target_link_libraries`, you can assume it's brought in implicitly via `find_dependency`.


- Alex


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


On May 26, 2015, 8:04 p.m., Hrvoje Senjan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123031/
> -----------------------------------------------------------
> 
> (Updated May 26, 2015, 8:04 p.m.)
> 
> 
> Review request for Build System, KDE Frameworks and Alex Merry.
> 
> 
> Repository: kross
> 
> 
> Description
> -------
> 
> Only search for public deps in cmake config.
> 
> Open question - shall we search *all* public deps, or minimal ones? e.g. target 1 link Qt5Core, target 2 links Qt5Gui. Do we search for both, or just Qt5Core?
> 
> 
> Diffs
> -----
> 
>   KF5KrossConfig.cmake.in e89cee2 
> 
> Diff: https://git.reviewboard.kde.org/r/123031/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Hrvoje Senjan
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20150718/5a601bf3/attachment.html>