================
@@ -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}")
----------------
ajordanr-google wrote:

This function is primarily here so that we don't have scope leak on modifying 
`CMAKE_REQUIRED_LIBRARIES` (which we want to restrict to only our given curses 
`.so`, and no other).

I'm not super sure how scope guarding works with how these `Find*` modules 
work, but I wanted to be sure that we don't actually modify 
CMAKE_REQUIRED_LIBRARIES outside of any other scope.

Not sure if that makes too much sense, but we _could_ move it out if you're 
okay with other libraries providing `acs_map`, not just the found 
`libcurses.so`.

https://github.com/llvm/llvm-project/pull/126810
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to