[all-commits] [llvm/llvm-project] 5acd37: [llvm][cmake] Make `install_symlink` workflow work...

John Ericson via All-commits all-commits at lists.llvm.org
Mon Jul 25 14:04:58 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 5acd376438a53747c84e38c8b69fc74a270da680
      https://github.com/llvm/llvm-project/commit/5acd376438a53747c84e38c8b69fc74a270da680
  Author: John Ericson <John.Ericson at Obsidian.Systems>
  Date:   2022-07-25 (Mon, 25 Jul 2022)

  Changed paths:
    M llvm/cmake/modules/AddLLVM.cmake
    M llvm/cmake/modules/LLVMInstallSymlink.cmake

  Log Message:
  -----------
  [llvm][cmake] Make `install_symlink` workflow work with absolute install dirs

If `CMAKE_INSTALL_BINDIR` is a different absolute path per project, as
it is with NixOS when we install every package to its own prefix, the
old way fails when the absolute path gets prepended with `CMAKE_INSTALL_PREFIX`.

Using `extend_path` from the install-time script isn't really possible, so we just make the caller responsible for making the path absolute instead.

Also fix one stray `bin` -> `CMAKE_INSTALL_BINDIR`

Reviewed By: sebastian-ne

Differential Revision: https://reviews.llvm.org/D101070




More information about the All-commits mailing list