[PATCH] CMake: add LLVM_INSTALL_DEV_FILES variable

[Hans Wennborg]

+chandlerc who didn't get cc'd automatically for some reason

On Fri, Aug 16, 2013 at 4:49 PM, Hans Wennborg <hans at chromium.org> wrote:
>
>   > Er, no?
>
>   OK, I've changed it.
>
>   > So, first note that this changes users of 'make install' in addition to users of cpack.
>
>   Right. I think my patch description actually only mentioned the 'install' target, because I don't think a lot of folks use cpack. Anyway, I'll expand the commit message that it affects both.
>
>   > There are two layers of this:
>   >
>   > 1) install development tools / resources useful for developing other *LLVM* projects (tblgen, etc)
>   >
>   > 2) install *anything* but executable binaries.
>   >
>   > I think these should be different. I think #1 should be default-off and turned on for users of the stand-alone Clang cmake build. It should be called something about "development".
>   >
>   > I think #2 should be default-on, because many many users of LLVM specifically want the core libraries. I think the best way is to negate the flag: make it INSTALL_TOOLCHAIN_BINARIES_ONLY or something, and when enabled, you only get the binaries used as part of a toolchain (llvm-{ar,nm,dwarfdump,...}, clang, clang-format, etc., but not opt or llc.
>
>   I don't think I'm following. You say #1 and #2 should be different. If #1 is development tools and resources (i.e. tablegen, headers and libraries), doesn't that overlap with #2 being anything but toolchain binaries?
>
>   I think my patch provides an good way to configure the install target. Currently, 'make install'  installs everything. I don't want to change that default. My patch just adds the possibility to exclude the "dev stuff", i.e. the libraries and headers. Isn't that pretty intuitive?
>
> http://llvm-reviews.chandlerc.com/D1428