[cfe-commits] [libcxx] r148309 - /libcxx/trunk/Makefile

[Howard Hinnant]

On Jan 17, 2012, at 7:30 PM, Bob Wilson wrote:

> 
> On Jan 17, 2012, at 9:05 AM, Howard Hinnant wrote:
> 
>> On Jan 17, 2012, at 11:59 AM, Bob Wilson wrote:
>> 
>>> On Jan 17, 2012, at 8:09 AM, Howard Hinnant wrote:
>>> 
>>>> Author: hhinnant
>>>> Date: Tue Jan 17 10:09:32 2012
>>>> New Revision: 148309
>>>> 
>>>> URL: http://llvm.org/viewvc/llvm-project?rev=148309&view=rev
>>>> Log:
>>>> Stop installing headers, this is now done by clang
>>> 
>>> Not anymore -- you just broke it.  We're using that do-installhdrs target that you removed (as noted in the comment).
>> 
>> Thanks for the catch Bob.  Just recommitted.  Please review.
>> 
>> Howard
> 
> Thanks, Howard.  I do have another comment….
> 
>> 
>>> 
>>>> 
>>>> Modified:
>>>> libcxx/trunk/Makefile
>>>> 
>>>> Modified: libcxx/trunk/Makefile
>>>> URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/Makefile?rev=148309&r1=148308&r2=148309&view=diff
>>>> ==============================================================================
>>>> --- libcxx/trunk/Makefile (original)
>>>> +++ libcxx/trunk/Makefile Tue Jan 17 10:09:32 2012
>>>> @@ -30,26 +30,7 @@
>>>> 
>>>> clean::
>>>> 
>>>> -installhdrs::
>>>> -
>>>> -	$(MAKE) HEADER_DIR=$(INSTALL_DIR)/usr/include do-installhdrs
>>>> -	$(MAKE) HEADER_DIR=$(INSTALL_DIR)/usr/lib do-installhdrs
>>>> -	$(MAKE) HEADER_DIR=$(INSTALL_DIR)/usr/clang-ide/lib do-installhdrs
>>>> -	$(MAKE) HEADER_DIR=$(INSTALL_DIR)/Developer/usr/lib do-installhdrs
>>>> -	$(MAKE) HEADER_DIR=$(INSTALL_DIR)/Developer/Platforms/iPhoneOS.platform/usr/lib do-installhdrs
> 
> It seems kind of awkward to have the make target named do-installhdrs.  I would prefer to drop the "do-" prefix.  But, to avoid rev lock with clang, it would be convenient for libc++ to support both "installhdrs" and "do-installhdrs" as synonyms, at least for some period of time.  We can change clang to use the "installhdrs" target, and after a while, libc++ can just remove the old "do-installhdrs" target.
> 
> How about inserting the following?
> 
> installhdrs:: do-installhdrs

Not a problem. Committed revision 148365.

Howard