<div class="gmail_quote">On Fri, Sep 24, 2010 at 10:57 AM, David Gobbi <span dir="ltr"><<a href="mailto:david.gobbi@gmail.com">david.gobbi@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
On Fri, Sep 24, 2010 at 8:45 AM, Marcus D. Hanwell<br>
<div class="im"><<a href="mailto:marcus.hanwell@kitware.com">marcus.hanwell@kitware.com</a>> wrote:<br>
</div><div class="im">> On Fri, Sep 24, 2010 at 10:29 AM, David Gobbi <<a href="mailto:david.gobbi@gmail.com">david.gobbi@gmail.com</a>> wrote:<br>
>><br>
</div><div class="im">>> But here is a caveat: setting PYTHON_MODULE_PREFIX might not be<br>
>> enough because there is a bug in CMake's Modules/FindPythonLibs.cmake.<br>
>> It does not actually set the prefix of the module, it only sets the<br>
>> prefix in the name of the "init" function.  I've fixed this bug in<br>
>> VTK's copy of FindPythonLibs.cmake, and someone should copy<br>
>> my changes from this file into CMake.<br>
><br>
> I can take a look at that.<br>
<br>
</div>Another thing that FindPythonLibs.cmake does not do is set the suffix<br>
to ".pyd" on Windows.  It didn't cross my mind when I was fixing the<br>
file, but since the ".dll" module suffix is officially deprecated by<br>
python, it might be good to have that in FindPythonLibs.cmake instead<br>
of scattered around various CMake files in various projects.<br>
<font color="#888888"><br></font></blockquote><div>I just pushed the updated FindPythonLibs into next for CMake, thanks for the fixes. I also fixed up ParaView's build system to take account of the new changes and will be looking at Titan next.</div>
<div><br></div><div>Thanks,</div><div><br></div><div>Marcus</div><div><div>--</div><div>Marcus D. Hanwell, Ph.D.</div><div>R&D Engineer, Kitware Inc.</div><div>(518) 881-4937</div></div></div>