Teach CMake to find versions of Python != 2.7

[zturner]

Summary

Hi Vadim,

Would you mind taking a look at this? My CMake-fu is only level 5, so if you have any suggestions on how to improve this, let me know.

Note that I'm claiming with this revision that LLDB itself works with Python versions other than 2.7. Only that CMake can set all the variables appropriately if you point it to a different version of CMake

[enlight]

cmake/modules/LLDBConfig.cmake

109

PYTHON_LIBRARY is used in this scope later on, so it needs to be explicitly set for both this scope and the parent scope, like so:

set (PYTHON_LIBRARY $<$<CONFIG:Debug>:${PYTHON_DEBUG_LIB}>$<$<NOT:$<CONFIG:Debug>>:${PYTHON_RELEASE_LIB}>)
set (PYTHON_LIBRARY ${PYTHON_LIBRARY} PARENT_SCOPE)

112–114

I think this belongs outside this function, along with include_directories below.

117

This command will never be executed because PYTHON_LIBRARY is only set in the parent scope. However, I'd prefer include_directories() wasn't here at all, let the caller of find_python_libs_windows do that instead so that the behavior is more similar to the built-in FindPythonLibs module (which means PYTHON_INCLUDE_DIRS should be made available in the parent scope, note the extra S).

131–133

Please move these into find_python_libs_windows(), as I'll need access to the debug/release paths in order to avoid printing out the generator expressions.

cmake/modules/LLDBStandalone.cmake

51–53

The comment no longer seems accurate.

[enlight]

cmake/modules/LLDBConfig.cmake

56

This will only set LLDB_DISABLE_PYTHON inside the function, you probably want to do set (LLDB_DISABLE_PYTHON 1 PARENT_SCOPE), or set (LLDB_DISABLE_PYTHON 1 CACHE INTERNAL "") instead.

73

Ditto.

[zturner]

Updated from Vadim's comments.

[enlight]

cmake/modules/LLDBConfig.cmake

113

You missed :P

140

Can we make this if (PYTHON_INCLUDE_DIRS) instead?

[enlight]

Hi Zach,

Can you close this issue please?

[zturner]

In the future, u sually the reviewer (you in this case) does this by hitting Accept in Phabricator. I can do it this time though, no worries