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