[PATCH] D42274: [Support] Remove the terminfo dependency and rely on TERM

[James Y Knight via Phabricator via llvm-commits]

jyknight added a comment.

I'm totally in favor of this, it looks like a great improvement.

But of course nbjoerg will complain because it will break all the people who run clang outputting to Tektronix 4012 graphics terminals which can't parse vt100/ansi escape sequence syntax.  And I don't have the will to argue the point.



================
Comment at: cmake/config-ix.cmake:154
     endif()
-    if(LLVM_ENABLE_TERMINFO)
-      set(HAVE_TERMINFO 0)
----------------
This stanza, and the option declaration above is still used by lldb's cmake file I believe.


Repository:
  rL LLVM

https://reviews.llvm.org/D42274