r252836 - [CMake] Setup an install component for libclang and c-index-test.

[Argyrios Kyrtzidis via cfe-commits]

Thanks for pointing it out.
r253001 sets up the destination to a different location only if ‘INTERNAL_INSTALL_PREFIX’ is set.

> On Nov 12, 2015, at 5:50 AM, Ismail Donmez <ismail at i10z.com> wrote:
> 
> Fixed in r252890
> 
> On Thu, Nov 12, 2015 at 2:17 PM, Ismail Donmez <ismail at i10z.com <mailto:ismail at i10z.com>> wrote:
> Hi,
> 
> On Thu, Nov 12, 2015 at 2:46 AM, Argyrios Kyrtzidis via cfe-commits <cfe-commits at lists.llvm.org <mailto:cfe-commits at lists.llvm.org>> wrote:
> --- cfe/trunk/tools/c-index-test/CMakeLists.txt (original)
> +++ cfe/trunk/tools/c-index-test/CMakeLists.txt Wed Nov 11 18:46:57 2015
> @@ -28,3 +28,12 @@ if (CLANG_HAVE_LIBXML)
>    include_directories(SYSTEM ${LIBXML2_INCLUDE_DIR})
>    target_link_libraries(c-index-test ${LIBXML2_LIBRARIES})
>  endif()
> +
> +install(TARGETS c-index-test
> +  RUNTIME DESTINATION local/bin
> +  COMPONENT c-index-test)
> 
> This doesn't look right, all of the llvm/clang install binaries under $PREFIX/bin, with this change c-index-test is going under $PREFIX/local/bin
> 
> ismail
> 
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20151112/752e1b89/attachment-0001.html>