[PATCH] D99484: Use `GNUInstallDirs` to support custom installation dirs.
    John Ericson via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Tue Apr 27 15:46:50 PDT 2021
    
    
  
Ericson2314 added inline comments.
================
Comment at: clang-tools-extra/clang-doc/tool/CMakeLists.txt:26
 install(FILES ../assets/index.js
-  DESTINATION share/clang
+  DESTINATION "${CMAKE_INSTALL_DATADIR}/clang"
   COMPONENT clang-doc)
----------------
compnerd wrote:
> Why are these quoted but other uses not?
I confess I have no clue when quoting is required or advisable with CMake. I started out converting things by hand and then did some auto-conversions, this must have been one of the by-hand ones.
Happy to normalize either way, just tell me which one.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D99484/new/
https://reviews.llvm.org/D99484
    
    
More information about the cfe-commits
mailing list