Review Request: Attica will not be found if installed under /usr/local/

[Guy Maurel]

> On 2010-09-17 18:52:17, Alexander Neundorf wrote:
> > /usr/local is part of the default search directories, so it should be found if installed there.
> > I just tested this here locally and it works for me.
> > 
> > If it is not found when installed in /usr/local/, then we have to check what is going wrong there.
> > 
> > Alex

mmmh!
I found at http://www.kitware.com/news/home/browse/168:
"FIND_LIBRARY and FIND_PATH now look in CMAKE_LIBRARY_PATH and CMAKE_INCLUDE_PATH environment variables in addition to and before the PATH environment variable. "
It is what I need, but nothing about a default value for CMAKE_LIBRARY_PATH and CMAKE_INCLUDE_PATH
Is it something the user *has* to do itself before running?


- Guy


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5375/#review7673
-----------------------------------------------------------


On 2010-09-17 18:08:15, Guy Maurel wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/5375/
> -----------------------------------------------------------
> 
> (Updated 2010-09-17 18:08:15)
> 
> 
> Review request for kdelibs.
> 
> 
> Summary
> -------
> 
> Some of the cmake/modules/Findxxx.cmake files have extended the search to the directory INCLUDE_INSTALL_DIR
> It is usefull to find the version on had installed just before. Many default installations are /usr/local/
> 
> My proposal is to do the same with Attica.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdelibs/cmake/modules/FindLibAttica.cmake 1175716 
> 
> Diff: http://svn.reviewboard.kde.org/r/5375/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Guy
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20100919/f880b0fb/attachment.htm>