Review Request 113703: Fix KRoss standalone build

Maarten De Meyer de.meyer.maarten at gmail.com
Thu Nov 7 13:45:28 UTC 2013



> On Nov. 7, 2013, 11:50 a.m., Alex Merry wrote:
> > tier3/kross/CMakeLists.txt, lines 29-31
> > <http://git.reviewboard.kde.org/r/113703/diff/1/?file=211380#file211380line29>
> >
> >     These shouldn't be needed; we should fix how KIO is generating KIOConfig.cmake first (it's not listing its dependencies properly).
> 
> Aurélien Gâteau wrote:
>     I think I fixed this yesterday in commit fd4771d7f11b847cc9ea9dbe1590f92893d16a12. Can you check you still need the find_package() calls?

With a fresh build they are no longer needed. Thanks.


- Maarten


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


On Nov. 7, 2013, 1:45 p.m., Maarten De Meyer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113703/
> -----------------------------------------------------------
> 
> (Updated Nov. 7, 2013, 1:45 p.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Repository: kdelibs
> 
> 
> Description
> -------
> 
> Make kross build standalone.
> 
> 
> Diffs
> -----
> 
>   tier3/kross/CMakeLists.txt 440edf2 
>   tier3/kross/src/modules/CMakeLists.txt 198cb04 
> 
> Diff: http://git.reviewboard.kde.org/r/113703/diff/
> 
> 
> Testing
> -------
> 
> Builds.
> Not added to superbuild.
> 
> 
> Thanks,
> 
> Maarten De Meyer
> 
>

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


More information about the Kde-frameworks-devel mailing list