FindPythonLibs should set PYTHON_INCLUDE_DIR instead of PYTHON_INCLUDE_PATH

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

FindPythonLibs should set PYTHON_INCLUDE_DIR instead of PYTHON_INCLUDE_PATH

Marcel Loose
Hi all,

In fact the subject line says it all. To be compliant with the naming
conventions proposed in the Modules/readme.txt file, FindPythonLibs
should set PYTHON_INCLUDE_DIR instead of PYTHON_INCLUDE_PATH.

To avoid breaking existing software, it would be best to simply just add
a definition for PYTHON_INCLUDE_DIR.

Best regards,
Marcel Loose.


_______________________________________________
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
Reply | Threaded
Open this post in threaded view
|

Re: FindPythonLibs should set PYTHON_INCLUDE_DIR instead of PYTHON_INCLUDE_PATH

Alexander Neundorf-3
On Monday 07 September 2009, Marcel Loose wrote:
> Hi all,
>
> In fact the subject line says it all. To be compliant with the naming
> conventions proposed in the Modules/readme.txt file, FindPythonLibs
> should set PYTHON_INCLUDE_DIR instead of PYTHON_INCLUDE_PATH.
>
> To avoid breaking existing software, it would be best to simply just add
> a definition for PYTHON_INCLUDE_DIR.

please put it in the bug tracker, ideally together with a patch :-)

Alex
_______________________________________________
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