Fixing a mis-use of the CMake install command.
authorChris Bieneman <beanz@apple.com>
Thu, 16 Apr 2015 16:56:18 +0000 (16:56 +0000)
committerChris Bieneman <beanz@apple.com>
Thu, 16 Apr 2015 16:56:18 +0000 (16:56 +0000)
commit1a0e0ea4fbf3838d9eb08aa10f43e94e1b0f0649
tree071b6ddc7ceba21353b72e3cfd96df4b7ba61f26
parentafc67405f8d3a755650a1cb4bb62ea76e21ed5fb
Fixing a mis-use of the CMake install command.

The CMake install command is defined as:

install(TARGETS targets... [EXPORT <export-name>]
        [[ARCHIVE|LIBRARY|RUNTIME|FRAMEWORK|BUNDLE|
          PRIVATE_HEADER|PUBLIC_HEADER|RESOURCE]
         [DESTINATION <dir>]
         [INCLUDES DESTINATION [<dir> ...]]
         [PERMISSIONS permissions...]
         [CONFIGURATIONS [Debug|Release|...]]
         [COMPONENT <component>]
         [OPTIONAL] [NAMELINK_ONLY|NAMELINK_SKIP]
        ] [...])

This means it can only take one parameter from the set of RUNTIME, LIBRARY, or ARCHIVE. If you set more than one of these it seems to gobble up the extra arguments and ignore the COMPONENT argument.

This adds a check to only set LIBRARY or ARCHIVE based on whether or not the library being built is shared.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@235113 91177308-0d34-0410-b5e6-96231b3b80d8
cmake/modules/AddLLVM.cmake