[cfe-commits] r167432 - /cfe/trunk/tools/c-index-test/Makefile

[Bob Wilson]

If you're trying to get c-index-test back into Apple builds, you'll need to do a bit more than this.  We now build with BUILD_CLANG_ONLY=YES, and that disables building c-index-test.  See the install-clang target in the top-level llvm Makefile and also the DIRS list in clang's tools/Makefile.  I suppose you can change those things so that c-index-test gets built even when BUILD_CLANG_ONLY=YES.  That will make the meaning of that build setting a little strange.

On Nov 5, 2012, at 6:12 PM, Argyrios Kyrtzidis <akyrtzi at gmail.com> wrote:

> Author: akirtzidis
> Date: Mon Nov  5 20:12:21 2012
> New Revision: 167432
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=167432&view=rev
> Log:
> [c-index-test] Do install c-index-test because it's not just for tests,
> it's also setup as 'INTERNAL_TOOL'.
> 
> Modified:
>    cfe/trunk/tools/c-index-test/Makefile
> 
> Modified: cfe/trunk/tools/c-index-test/Makefile
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/tools/c-index-test/Makefile?rev=167432&r1=167431&r2=167432&view=diff
> ==============================================================================
> --- cfe/trunk/tools/c-index-test/Makefile (original)
> +++ cfe/trunk/tools/c-index-test/Makefile Mon Nov  5 20:12:21 2012
> @@ -17,9 +17,6 @@
> # No plugins, optimize startup time.
> TOOL_NO_EXPORTS = 1
> 
> -# Don't install this. It is used for tests.
> -NO_INSTALL = 1
> -
> # Include this here so we can get the configuration of the targets that have
> # been configured for construction. We have to do this early so we can set up
> # LINK_COMPONENTS before including Makefile.rules
> 
> 
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits