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

[Argyrios Kyrtzidis]

On Nov 6, 2012, at 8:31 AM, Bob Wilson <bob.wilson at apple.com> wrote:

> 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.

Thanks for letting me know. I did the updates in llvm r167484 and clang 167478.

> 
> 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
>