Author: jdevlieghere Date: Wed Jul 17 12:36:20 2019 New Revision: 366363 URL: http://llvm.org/viewvc/llvm-project?rev=366363&view=rev Log: [CMake] Remove duplicated logic to find Python when doing a standalone build
I'm pretty sure there's no need to have this logic living in LLDBStandalone. It doesn't appear anything in LLVM depends on this, and We always go through LLDBConfig.cmake which has the canonical way to find the Python libs and interpreter for LLDB. Differential revision: https://reviews.llvm.org/D64821 Modified: lldb/trunk/cmake/modules/LLDBStandalone.cmake Modified: lldb/trunk/cmake/modules/LLDBStandalone.cmake URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/cmake/modules/LLDBStandalone.cmake?rev=366363&r1=366362&r2=366363&view=diff ============================================================================== --- lldb/trunk/cmake/modules/LLDBStandalone.cmake (original) +++ lldb/trunk/cmake/modules/LLDBStandalone.cmake Wed Jul 17 12:36:20 2019 @@ -77,18 +77,6 @@ include(HandleLLVMOptions) include(CheckAtomic) include(LLVMDistributionSupport) -set(Python_ADDITIONAL_VERSIONS 3.7 3.6 3.5 2.7) -if (PYTHON_EXECUTABLE STREQUAL "") - include(FindPythonInterp) - if( NOT PYTHONINTERP_FOUND ) - message(FATAL_ERROR - "Unable to find Python interpreter, required for builds and testing. - Please install Python or specify the PYTHON_EXECUTABLE CMake variable.") - endif() -else() - message(STATUS "Found PythonInterp: ${PYTHON_EXECUTABLE}") -endif() - set(PACKAGE_VERSION "${LLVM_PACKAGE_VERSION}") set(LLVM_INCLUDE_TESTS ON CACHE INTERNAL "") _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits