Review Request 126896: Make sure the prefix is looked-up

[Friedrich W. H. Kossebau]

> On July 23, 2016, 7 p.m., Martin Tobias Holmedahl Sandsmark wrote:
> > isn't this wrong? it leads to cmake looking in /usr/include, and if it finds something there it tries to use it (e. g. execinfo.h, which pulls in the host glibc, breaking everything).

Yes, this also broke things for most people trying to build Marble for Android. Going to before this commit fixed things for them.

Proposal how to fix this by keeping host and target dirs (also for cmake config files) more cleanly separated here: https://phabricator.kde.org/D3646


- Friedrich W. H.


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


On Jan. 28, 2016, 11:47 a.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126896/
> -----------------------------------------------------------
> 
> (Updated Jan. 28, 2016, 11:47 a.m.)
> 
> 
> Review request for Extra Cmake Modules and Andreas Cord-Landwehr.
> 
> 
> Repository: extra-cmake-modules
> 
> 
> Description
> -------
> 
> I've tried to compile ki18n against libintl-lite. This was required to be able to find it in our prefix.
> 
> I can't but expect that other libraries would have a similar problem.
> 
> The only reason why it used to work, is because all libraries we're including, provide `*Config.cmake` files, which don't respect this setting.
> 
> 
> Diffs
> -----
> 
>   toolchain/Android.cmake 81ded8f 
> 
> Diff: https://git.reviewboard.kde.org/r/126896/diff/
> 
> 
> Testing
> -------
> 
> Now it fails somwhere else when building ki18n.
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20161211/45c252ae/attachment.html>