[Lldb-commits] [lldb] [lldb] Add terminfo dependency for ncurses support (PR #126810)

Jonas Devlieghere via lldb-commits lldb-commits at lists.llvm.org
Fri Feb 14 16:24:00 PST 2025


================
@@ -2,23 +2,54 @@
 # FindCursesAndPanel
 # -----------
 #
-# Find the curses and panel library as a whole.
+# Find the curses, terminfo, and panel library as a whole.
 
-if(CURSES_INCLUDE_DIRS AND CURSES_LIBRARIES AND PANEL_LIBRARIES)
+include(CMakePushCheckState)
+
+function(lldb_check_curses_tinfo CURSES_LIBRARIES CURSES_HAS_TINFO)
+  cmake_reset_check_state()
+  set(CMAKE_REQUIRED_LIBRARIES "${CURSES_LIBRARIES}")
----------------
JDevlieghere wrote:

Could we eliminate this if we moved and inlined the `check_symbol_exists` in the if-check on line 28? Presumably at that point we know curses was found? 

https://github.com/llvm/llvm-project/pull/126810


More information about the lldb-commits mailing list