<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Sep 16, 2015 at 9:41 AM, Brad King <span dir="ltr"><<a href="mailto:brad.king@kitware.com" target="_blank">brad.king@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 09/16/2015 11:39 AM, Brad King wrote:<br>
> On 09/16/2015 10:00 AM, David Gobbi wrote:<br>
>> this new patch only changes the search for the include dirs.<br>
><br>
> Thanks.  Rather than calling find_path twice, the first call could<br>
> just use HINTS instead of PATHS.  HINTS are meant for this use case:<br>
> searching paths detected from the system in some manner.  They are<br>
> searched before the generic system and environment paths.  Does that<br>
> work for your use case?<br>
><br>
> Also, why does the second call not use the same list of suffixes?<br>
<br>
</span>Meanwhile I split out the OS X framework path fix:<br>
<br>
 FindPythonLibs: Fix OS X framework include directory search path<br>
 <a href="http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=ea2db3bb" rel="noreferrer" target="_blank">http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=ea2db3bb</a><br>
<br>
Please check that it looks correct on its own.</blockquote><div><br></div><div>Thanks, looks fine.  I've modified my patch so that it only calls find_path once.</div><div>It uses HINTS to search for the includes in the same prefix as where the library</div><div>was found, which is probably the best place to start.  I've left the framework dirs</div><div>in PATHS because we don't necessarily want them to have priority, we just</div><div>want to have them in the search path.</div><div><br></div><div> - David   </div></div><br></div></div>