[PATCH] D85820: Use find_library for ncurses
Harmen Stoppels via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 12 12:40:31 PDT 2020
haampie added inline comments.
================
Comment at: llvm/cmake/config-ix.cmake:151-155
set(HAVE_TERMINFO 0)
- foreach(library terminfo tinfo curses ncurses ncursesw)
- string(TOUPPER ${library} library_suffix)
- check_library_exists(${library} setupterm "" HAVE_TERMINFO_${library_suffix})
- if(HAVE_TERMINFO_${library_suffix})
- set(HAVE_TERMINFO 1)
- set(TERMINFO_LIBS "${library}")
- break()
- endif()
- endforeach()
+ find_library(FIND_TERMINFO NAMES terminfo tinfo curses ncurses ncursesw)
+ if(FIND_TERMINFO)
+ set(HAVE_TERMINFO 1)
+ set(TERMINFO_LIBS "${FIND_TERMINFO}")
----------------
JDevlieghere wrote:
> You could simplify this a little by using `TERMINFO_LIBS` directly.
Thanks, I made it even simpler
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85820/new/
https://reviews.llvm.org/D85820
More information about the llvm-commits
mailing list