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

Christophe Giboudeaux cgiboudeaux at gmx.com
Wed Mar 18 20:31:56 UTC 2015



> On March 18, 2015, 7:40 p.m., Christophe Giboudeaux wrote:
> > KF5KrossConfig.cmake.in, line 9
> > <https://git.reviewboard.kde.org/r/123031/diff/1/?file=355503#file355503line9>
> >
> >     Are you sure ?
> >     
> >     ./ui/actioncollectionview.h:27:#include <QWidget>
> 
> Hrvoje Senjan wrote:
>     this is now similar (reverse) as with kpeople review request. do we find the public deps for all targets, or for the 'minimal' one?

Maybe I misunderstood the find_dependency goal but it sounds to me that if you #include a file in your installed header, it becomes de facto a dependency


- Christophe


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


On March 18, 2015, 8:13 p.m., Hrvoje Senjan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123031/
> -----------------------------------------------------------
> 
> (Updated March 18, 2015, 8:13 p.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Repository: kross
> 
> 
> Description
> -------
> 
> Only search for public deps in cmake config
> 
> 
> 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-frameworks-devel/attachments/20150318/ef0a4676/attachment.html>


More information about the Kde-frameworks-devel mailing list