[Lldb-commits] [clang] [compiler-rt] [lldb] [llvm] [Support] Remove terminfo dependency (PR #92865)

[Jonas Devlieghere via lldb-commits]

JDevlieghere wrote:

`LLDB_CURSES_LIBS` is scoped to lldbCore, which the EditLine unit test isn't linking. If its also necessary in Host, then we should hoist it up and link both libraries against it.

Anyway, that also means my initial assessment was wrong. I looked for the header include but the code in our editline wrapper forward declares `setupterm` which explains why I missed it. Maybe we should reconsider? Anyway, if I cannot figure this out in the next hour or so, I think we should revert this to get our bots green again. 

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