[PATCH] D46521: [llvm] cmake: resolve symlinks in LLVMConfig.cmake

Michał Górny via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 8 09:08:50 PDT 2018


mgorny added a comment.

…at the cost of breaking CMake, and having to hack LLVM and CMake to make it work again. And I'm pretty sure sooner or later this realpath use will break somebody's use case. Think of a simple example of someone using a temporary symlink in path to relocate LLVM install. By using realpath, you're effectively replacing the canonical location with possibly temporary current location that may no longer be valid afterwards.


Repository:
  rL LLVM

https://reviews.llvm.org/D46521





More information about the llvm-commits mailing list