[prev in list] [next in list] [prev in thread] [next in thread] 

List:       cmake
Subject:    Re: [CMake] find_library search orders with CMAKE_PREFIX_PATH & CMAKE_FIND_ROOT_PATH
From:       Nicolas Tisserand <nt () manctl ! com>
Date:       2012-05-24 5:04:54
Message-ID: CAD=O30m4ZhGyL-1+YmZRb8NKWUGDAsYyzN8By1j0gJMhwRUhBw () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


On Wed, May 23, 2012 at 11:29 AM, David Cole <david.cole@kitware.com> wrote:

> Regarding the loop swapping you ask about in your original post:
>
> I know we've discussed swapping those loops before on this list, quite to
> exhaustion.


I'm sorry to read that... now that I raised the topic again.


> The bottom line is: we will not be changing those loops so as to preserve
> existing behavior. If we do add code to swap those loops, then it would
> have to be conditionally activated, only in response to people requesting
> it by explicitly setting a variable (or some equivalent mechanism).
>

That sound perfectly fine to me. In fact, the fact that the CMake team is
doing a great job at not breaking existing build systems when shipping new
versions is one of the reasons why I use CMake.


> I don't know how far back this discussion was, but it's in the archives.
> (If anybody can provide a pointer to the exact thread, that would be
> great...)
>

Was a consensus reached, back then?

I would very much like to see such an addition to CMake.

As proposed in my previous email, an additional "FIRST" value to the
CMAKE_FIND_ROOT_PATH_MODE_{PROGRAM,INCLUDE,LIBRARY} {ONLY,NEVER,BOTH} set
would perfectly fit the bill, in my case.

Would that be hard to do?

Would it be considered as a welcome addition?

Best,
Nicolas

[Attachment #5 (text/html)]

<div class="gmail_quote">On Wed, May 23, 2012 at 11:29 AM, David Cole <span \
dir="ltr">&lt;<a href="mailto:david.cole@kitware.com" \
target="_blank">david.cole@kitware.com</a>&gt;</span> wrote:<br><blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex">

<div>Regarding the loop swapping you ask about in your original \
post:</div><div><br></div>I know we&#39;ve discussed swapping those loops before on \
this list, quite to exhaustion.</blockquote><div><br></div><div>I&#39;m sorry to read \
that... now that I raised the topic again.</div>

<div>  </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px \
#ccc solid;padding-left:1ex"><div>The bottom line is: we will not be changing those \
loops so as to preserve existing behavior. If we do add code to swap those loops, \
then it would have to be conditionally activated, only in response to people \
requesting it by explicitly setting a variable (or some equivalent mechanism).</div>

</blockquote><div><br></div><div>That sound perfectly fine to me. In fact, the fact \
that the CMake team is doing a great job at not breaking existing build systems when \
shipping new versions is one of the reasons why I use CMake.</div>

<div>  </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px \
#ccc solid;padding-left:1ex"><div>I don&#39;t know how far back this discussion was, \
but it&#39;s in the archives. (If anybody can provide a pointer to the exact thread, \
that would be great...)<br>

</div></blockquote><div><br></div><div>Was a consensus reached, back \
then?</div><div><br></div><div>I would very much like to see such an addition to \
CMake.</div><div><br></div><div>As proposed in my previous email, an additional \
&quot;<span style>FIRST&quot;</span>  value to the  <span \
style>CMAKE_FIND_ROOT_PATH_MODE_{</span><span style>PROGRAM,INCLUDE,LIBRARY} \
{ONLY,NEVER,BOTH} set would perfectly fit the bill, in my case.</span></div>

<div><span style><br></span></div><div><span style>Would that be hard to \
do?</span></div><div><span style><br></span></div><div><span style>Would it be \
considered as a welcome addition?</span></div><div><span style><br></span></div>

<div><font color="#222222" face="arial, sans-serif">Best,</font></div><div><font \
color="#222222" face="arial, sans-serif">Nicolas</font></div></div>



--

Powered by www.kitware.com

Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html

Please keep messages on-topic and check the CMake FAQ at: http://www.cmake.org/Wiki/CMake_FAQ

Follow this link to subscribe/unsubscribe:
http://www.cmake.org/mailman/listinfo/cmake

[prev in list] [next in list] [prev in thread] [next in thread]