<div dir="ltr">Yeah, this isn't what we want. Go ahead and revert.</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 26, 2015 at 9:37 AM, Eric Fiselier <span dir="ltr"><<a href="mailto:eric@efcs.ca" target="_blank">eric@efcs.ca</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ping.<br>
<div class="HOEnZb"><div class="h5"><br>
On Wed, Mar 18, 2015 at 10:19 PM, Eric Fiselier <<a href="mailto:eric@efcs.ca">eric@efcs.ca</a>> wrote:<br>
> Can you explain why you need this?<br>
><br>
> Previously these header files get installed under the c++ standard<br>
> library include directory (typically <root>/include/c++/v1) which is<br>
> not a default directory.<br>
> Now they get installed along what is likely a default system header<br>
> search path. This is probably not what we want.<br>
><br>
> For example, I ran into trouble trying to compile a program that uses<br>
> GNU's libunwind with g++ and libstdc++. The build would pick up<br>
> libc++abi's "unwind.h" which was installed in "/usr/local/include" and<br>
> then it would link to GNU's libunwind which resulted in a bunch of<br>
> undefined symbols.<br>
><br>
> I think we should revert this and go back to the old behavior of not<br>
> installing these headers until we can come up with a better way to do<br>
> this.<br>
><br>
> /Eric<br>
><br>
><br>
><br>
> On Thu, Feb 5, 2015 at 5:46 PM, Greg Fitzgerald <<a href="mailto:garious@gmail.com">garious@gmail.com</a>> wrote:<br>
>> Author: garious<br>
>> Date: Thu Feb  5 16:46:43 2015<br>
>> New Revision: 228351<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=228351&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=228351&view=rev</a><br>
>> Log:<br>
>> Install header files<br>
>><br>
>> Modified:<br>
>>     libcxxabi/trunk/CMakeLists.txt<br>
>><br>
>> Modified: libcxxabi/trunk/CMakeLists.txt<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/libcxxabi/trunk/CMakeLists.txt?rev=228351&r1=228350&r2=228351&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/libcxxabi/trunk/CMakeLists.txt?rev=228351&r1=228350&r2=228351&view=diff</a><br>
>> ==============================================================================<br>
>> --- libcxxabi/trunk/CMakeLists.txt (original)<br>
>> +++ libcxxabi/trunk/CMakeLists.txt Thu Feb  5 16:46:43 2015<br>
>> @@ -315,3 +315,10 @@ if(NOT LIBCXXABI_ENABLE_SHARED)<br>
>>  else()<br>
>>    add_subdirectory(test)<br>
>>  endif()<br>
>> +<br>
>> +install(DIRECTORY include/<br>
>> +  DESTINATION include<br>
>> +  FILES_MATCHING<br>
>> +    PATTERN "*.h"<br>
>> +    PATTERN ".svn" EXCLUDE<br>
>> +)<br>
>><br>
>><br>
>> _______________________________________________<br>
>> cfe-commits mailing list<br>
>> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div>