Tim Armstrong has posted comments on this change.

Change subject: Disable terminfo dependency for LLVM
......................................................................


Patch Set 1:

libncurses should be available pretty much everywhere, but libtinfo takes 
precedence so it's a bit weird.

It also makes zero sense to have LLVM's JITs depend on ncurses/tinfo (generally 
you wouldn't want all that stuff linked into the process) but that's the way 
they factored their library. It would be great if we could turn it on for clang 
and off otherwise, but that doesn't seem to be an option.

-- 
To view, visit http://gerrit.cloudera.org:8080/2860
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3e4fc73d4409592d55beb45a622e9f4a7deb5317
Gerrit-PatchSet: 1
Gerrit-Project: Toolchain
Gerrit-Branch: master
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Casey Ching <[email protected]>
Gerrit-Reviewer: Henry Robinson <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-HasComments: No

Reply via email to