[Lldb-commits] [PATCH] D51999: build: add libedit to include paths

[Tatyana Krasnukha via Phabricator via lldb-commits]

tatyana-krasnukha updated this revision to Diff 166258.
tatyana-krasnukha added a comment.

Unfortunately, doing this only on `lldbHost` is not enough, because CMake 
doesn't propagate include directories to dependent projects. I added these 
lines to every project that includes "Editline.h".


Repository:
  rLLDB LLDB

https://reviews.llvm.org/D51999

Files:
  source/Core/CMakeLists.txt
  source/Host/CMakeLists.txt
  source/Interpreter/CMakeLists.txt
  unittests/Editline/CMakeLists.txt


Index: unittests/Editline/CMakeLists.txt
===================================================================
--- unittests/Editline/CMakeLists.txt
+++ unittests/Editline/CMakeLists.txt
@@ -6,3 +6,7 @@
     lldbHost
     lldbUtility
   )
+
+if (NOT LLDB_DISABLE_LIBEDIT)
+  target_include_directories(EditlineTests PUBLIC ${libedit_INCLUDE_DIRS})
+endif()
Index: source/Interpreter/CMakeLists.txt
===================================================================
--- source/Interpreter/CMakeLists.txt
+++ source/Interpreter/CMakeLists.txt
@@ -55,3 +55,7 @@
   LINK_COMPONENTS
     Support
   )
+
+if (NOT LLDB_DISABLE_LIBEDIT)
+  target_include_directories(lldbInterpreter PUBLIC ${libedit_INCLUDE_DIRS})
+endif()
\ No newline at end of file
Index: source/Host/CMakeLists.txt
===================================================================
--- source/Host/CMakeLists.txt
+++ source/Host/CMakeLists.txt
@@ -174,3 +174,7 @@
     Object
     Support
   )
+
+if (NOT LLDB_DISABLE_LIBEDIT)
+  target_include_directories(lldbHost PUBLIC ${libedit_INCLUDE_DIRS})
+endif()
Index: source/Core/CMakeLists.txt
===================================================================
--- source/Core/CMakeLists.txt
+++ source/Core/CMakeLists.txt
@@ -80,3 +80,7 @@
 # Needed to properly resolve references in a debug build.
 # TODO: Remove once we have better layering
 set_target_properties(lldbCore PROPERTIES LINK_INTERFACE_MULTIPLICITY 4)
+
+if (NOT LLDB_DISABLE_LIBEDIT)
+  target_include_directories(lldbCore PUBLIC ${libedit_INCLUDE_DIRS})
+endif()



Index: unittests/Editline/CMakeLists.txt
===================================================================
--- unittests/Editline/CMakeLists.txt
+++ unittests/Editline/CMakeLists.txt
@@ -6,3 +6,7 @@
     lldbHost
     lldbUtility
   )
+
+if (NOT LLDB_DISABLE_LIBEDIT)
+  target_include_directories(EditlineTests PUBLIC ${libedit_INCLUDE_DIRS})
+endif()
Index: source/Interpreter/CMakeLists.txt
===================================================================
--- source/Interpreter/CMakeLists.txt
+++ source/Interpreter/CMakeLists.txt
@@ -55,3 +55,7 @@
   LINK_COMPONENTS
     Support
   )
+
+if (NOT LLDB_DISABLE_LIBEDIT)
+  target_include_directories(lldbInterpreter PUBLIC ${libedit_INCLUDE_DIRS})
+endif()
\ No newline at end of file
Index: source/Host/CMakeLists.txt
===================================================================
--- source/Host/CMakeLists.txt
+++ source/Host/CMakeLists.txt
@@ -174,3 +174,7 @@
     Object
     Support
   )
+
+if (NOT LLDB_DISABLE_LIBEDIT)
+  target_include_directories(lldbHost PUBLIC ${libedit_INCLUDE_DIRS})
+endif()
Index: source/Core/CMakeLists.txt
===================================================================
--- source/Core/CMakeLists.txt
+++ source/Core/CMakeLists.txt
@@ -80,3 +80,7 @@
 # Needed to properly resolve references in a debug build.
 # TODO: Remove once we have better layering
 set_target_properties(lldbCore PROPERTIES LINK_INTERFACE_MULTIPLICITY 4)
+
+if (NOT LLDB_DISABLE_LIBEDIT)
+  target_include_directories(lldbCore PUBLIC ${libedit_INCLUDE_DIRS})
+endif()

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits