This is an archive of the discontinued LLVM Phabricator instance.

Patch to specify the library containing terminfo symbols
Needs ReviewPublic

Authored by glennpj on Feb 18 2020, 1:56 PM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

When compiling LLVM on Spack, I saw library linking to my system libtinfo rather than the spack built ncurses. This is because the search path in config-ix.cmake list tinfo first.

The atttached patch creates a new variable to use to specify the library that contains the tinfo symbols, and then makes sure that it is first in the list. This may need to have a conditional relationship with LLVM_ENABLE_TERMINFO but that is not done in this patch.

Diff Detail

Event Timeline

glennpj created this revision.Feb 18 2020, 1:56 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 18 2020, 1:56 PM