[all-commits] [llvm/llvm-project] a52173: Use find_library for ncurses

Harmen Stoppels via All-commits all-commits at lists.llvm.org
Mon Aug 17 19:56:41 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: a52173a3e56553d7b795bcf3cdadcf6433117107
      https://github.com/llvm/llvm-project/commit/a52173a3e56553d7b795bcf3cdadcf6433117107
  Author: Harmen Stoppels <harmenstoppels at gmail.com>
  Date:   2020-08-17 (Mon, 17 Aug 2020)

  Changed paths:
    M compiler-rt/cmake/config-ix.cmake
    M compiler-rt/lib/xray/tests/CMakeLists.txt
    M lldb/source/Core/CMakeLists.txt
    M llvm/cmake/config-ix.cmake
    M llvm/include/llvm/Config/config.h.cmake
    M llvm/lib/Support/CMakeLists.txt
    M llvm/lib/Support/Unix/Process.inc
    M llvm/utils/gn/secondary/llvm/include/llvm/Config/BUILD.gn

  Log Message:
  -----------
  Use find_library for ncurses

Currently it is hard to avoid having LLVM link to the system install of
ncurses, since it uses check_library_exists to find e.g. libtinfo and
not find_library or find_package.

With this change the ncurses lib is found with find_library, which also
considers CMAKE_PREFIX_PATH. This solves an issue for the spack package
manager, where we want to use the zlib installed by spack, and spack
provides the CMAKE_PREFIX_PATH for it.

This is a similar change as https://reviews.llvm.org/D79219, which just
landed in master.

Differential revision: https://reviews.llvm.org/D85820




More information about the All-commits mailing list