[Lldb-commits] [lldb] r234319 - Don't compile scripts directory if LLDB_DISABLE_PYTHON is set

Pavel Labath labath at google.com
Tue Apr 7 07:55:11 PDT 2015


Author: labath
Date: Tue Apr  7 09:55:11 2015
New Revision: 234319

URL: http://llvm.org/viewvc/llvm-project?rev=234319&view=rev
Log:
Don't compile scripts directory if LLDB_DISABLE_PYTHON is set

Summary:
What looks like a typo has caused the scripts/Python directory to be compiled on non-Windows
platforms even with LLDB_DISABLE_PYTHON, which failed if Python.h was unavaiable. This changes
the condition to avoid compilation if LLDB_DISABLE_PYTHON is set.

Test Plan: Remove Python.h, verify compilation is successful.

Reviewers: zturner

Subscribers: lldb-commits

Differential Revision: http://reviews.llvm.org/D8855

Modified:
    lldb/trunk/CMakeLists.txt

Modified: lldb/trunk/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/CMakeLists.txt?rev=234319&r1=234318&r2=234319&view=diff
==============================================================================
--- lldb/trunk/CMakeLists.txt (original)
+++ lldb/trunk/CMakeLists.txt Tue Apr  7 09:55:11 2015
@@ -4,7 +4,7 @@ include(cmake/modules/AddLLDB.cmake)
 
 #add_subdirectory(include)
 add_subdirectory(docs)
-if (NOT CMAKE_SYSTEM_NAME MATCHES "Windows" OR NOT LLDB_DISABLE_PYTHON)
+if (NOT CMAKE_SYSTEM_NAME MATCHES "Windows" AND NOT LLDB_DISABLE_PYTHON)
   add_subdirectory(scripts)
 endif ()
 add_subdirectory(source)





More information about the lldb-commits mailing list