[PATCH] D100810: Use `GNUInstallDirs` to support custom installation dirs. -- LLVM

John Ericson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 3 21:13:50 PDT 2021


Ericson2314 added a comment.

@nib Sorry for not catching this myself. I must admit the error does confuse me. I have an macOS machine I can ssh to, and yes, if if you had some tips on how to reproduce there I would greatly appreciate them.

Thanks.



================
Comment at: llvm/docs/CMake.rst:270
+  Defaults to ``share/man``.
+
 .. _LLVM-related variables:
----------------
compnerd wrote:
> I'm kinda torn on this.  The variables here are all CMake standard and we should redirect users to CMake.  Although, this is following the existing pattern, so seems reasonable.
Yeah maybe we can redo the section after. "rarely used" is a funny title; shouldn't we be mentioning he most notable CMake variables if any at all?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D100810/new/

https://reviews.llvm.org/D100810



More information about the llvm-commits mailing list