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

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 24 09:07:05 PDT 2018


thakis added a comment.

Background: This was added in https://reviews.llvm.org/rL187874 ; from what I understand because someone google-internally who really doesn't like colored output complained that their existing setup for disabling colors in all programs wasn't honored by clang. It regressed color output without curses until r322962 finally restored that functionality. I agree that undoing this would be a good change in general, but said person might be unhappy about it. Maybe we can at least flip the default to false, but just removing it altogether if that's what gcc does these days seems very reasonable to me.


Repository:
  rL LLVM

https://reviews.llvm.org/D42274





More information about the llvm-commits mailing list