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

[Bob Wilson]

Oh right.   The "installhdrs" target has special meaning for Apple builds.  How about "installheaders"?

On Jan 18, 2012, at 8:10 AM, Howard Hinnant wrote:

> On Jan 17, 2012, at 7:35 PM, Howard Hinnant wrote:
> 
>> 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.
> 
> It turns out that this is a problem after all.  I have to revert this change.
> 
> Howard
>