vzakhari added inline comments.
================
Comment at: clang/lib/Headers/CMakeLists.txt:176
install(
- DIRECTORY ${output_dir}
+ FILES ${install_files}
DESTINATION ${header_install_dir}
----------------
vzakhari wrote:
> This is going to flatten the headers' install directory structure. install()
> will put the files just by their name.
>
> I had a local fix that use install() with RENAME, and it worked. I can try
> to find it. Basically, we need to keep two lists:
> 1. The same as this ${install_files}
> 2. New list install_as, which holds all ${src} values passed to
> copy_header_to_output_dir()
>
> Walking the two lists together we may use install(FILES
> ${elt_from_install_files} RENAME ${elt_from_install_as} ...)
Correction.
> New list install_as, which holds all ${src} values passed to
> copy_header_to_output_dir()
Should be
New list install_as, which holds all ${file} values passed to
copy_header_to_output_dir()
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61220/new/
https://reviews.llvm.org/D61220
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
