Lgtm<br><div class="gmail_quote">On Tue, Apr 7, 2015 at 2:53 AM Pavel Labath <<a href="mailto:labath@google.com">labath@google.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi zturner,<br>
<br>
What looks like a typo has caused the scripts/Python directory to be compiled on non-Windows<br>
platforms even with LLDB_DISABLE_PYTHON, which failed if Python.h was unavaiable. This changes<br>
the condition to avoid compilation if LLDB_DISABLE_PYTHON is set.<br>
<br>
<a href="http://reviews.llvm.org/D8855" target="_blank">http://reviews.llvm.org/D8855</a><br>
<br>
Files:<br>
  CMakeLists.txt<br>
<br>
Index: CMakeLists.txt<br>
==============================<u></u>==============================<u></u>=======<br>
--- CMakeLists.txt<br>
+++ CMakeLists.txt<br>
@@ -4,7 +4,7 @@<br>
<br>
 #add_subdirectory(include)<br>
 add_subdirectory(docs)<br>
-if (NOT CMAKE_SYSTEM_NAME MATCHES "Windows" OR NOT LLDB_DISABLE_PYTHON)<br>
+if (NOT CMAKE_SYSTEM_NAME MATCHES "Windows" AND NOT LLDB_DISABLE_PYTHON)<br>
   add_subdirectory(scripts)<br>
 endif ()<br>
 add_subdirectory(source)<br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/<u></u>settings/panel/<u></u>emailpreferences/</a><br>
</blockquote></div>