<div dir="ltr">gen_link_script.py is missing a license header.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 22, 2015 at 5:39 PM, Alexey Samsonov via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Seems to work now, thanks for the quick fix!</div><div class="gmail_extra"><div><div class="h5"><br><div class="gmail_quote">On Thu, Oct 22, 2015 at 1:57 PM, 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"><div dir="ltr">Hi Alexey,<div><br></div><div>Please confirm that this works for you now after r251063. Sorry for the breakage.</div><span><font color="#888888"><div><br></div><div>/Eric</div></font></span></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 22, 2015 at 9:23 AM, Alexey Samsonov <span dir="ltr"><<a href="mailto:vonosmas@gmail.com" target="_blank">vonosmas@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">After this change I am unable to configure libcxx as external project from compiler-rt with extra build flags.<div>The problem is gen_link_script.py is invoked with incorrect number of arguments: LIBCXX_CXX_ABI_LIBNAME is automatically deduced to be "none",</div><div>and LIBCXX_CXX_ABI_LIBRARY which is passed to gen_link_script.py is then empty.</div><div><br></div><div>I can suppress this by using condition</div><div>  if (LIBCXX_ENABLE_ABI_LINKER_SCRIPT AND LIBCXX_CXX_ABI_LIBRARY)<br></div><div>in libcxx/lib/CMakeLists.txt, but I don't think this fix is clean enough. I can also pass -DLIBCXX_ENABLE_ABI_LINKER_SCRIPT=OFF when I configure libc++,</div><div>but probably this should be deduced automatically...</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div>On Wed, Oct 21, 2015 at 5:44 PM, Eric Fiselier via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><div dir="ltr"><p>Thanks. I'll look to see if any other LLVM projects have bumped the CMake version to 3 yet. If so that's the direction I will head.</p><div><div><p><br></p><p><br></p><p><br></p>
<div class="gmail_quote">On Oct 20, 2015 9:39 PM, "Hahnfeld, Jonas" <<a href="mailto:Hahnfeld@itc.rwth-aachen.de" target="_blank">Hahnfeld@itc.rwth-aachen.de</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> -----Original Message-----<br>
> From: cfe-commits [mailto:<a href="mailto:cfe-commits-bounces@lists.llvm.org" target="_blank">cfe-commits-bounces@lists.llvm.org</a>] On Behalf<br>
> Of Eric Fiselier via cfe-commits<br>
> Sent: Wednesday, October 14, 2015 9:54 PM<br>
> To: <a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
> Subject: [libcxx] r250319 - [libcxx] Make it drastically simpler to link<br>
> libc++.<br>
><br>
> Author: ericwf<br>
> Date: Wed Oct 14 14:54:03 2015<br>
> New Revision: 250319<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=250319&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=250319&view=rev</a><br>
> Log:<br>
> [libcxx] Make it drastically simpler to link libc++.<br>
[...]<br>
>  if (LIBCXX_INSTALL_LIBRARY)<br>
>    install(TARGETS cxx<br>
>      LIBRARY DESTINATION lib${LIBCXX_LIBDIR_SUFFIX} COMPONENT libcxx<br>
>      ARCHIVE DESTINATION lib${LIBCXX_LIBDIR_SUFFIX} COMPONENT libcxx<br>
>      )<br>
> +  # NOTE: This install command must go after the cxx install command<br>
> + otherwise  # it will not be executed after the library symlinks are<br>
> installed.<br>
> +  if (LIBCXX_ENABLE_ABI_LINKER_SCRIPT)<br>
> +    install(FILES "$<TARGET_LINKER_FILE:cxx>"<br>
> +      DESTINATION lib${LIBCXX_LIBDIR_SUFFIX}<br>
> +      COMPONENT libcxx)<br>
> +  endif()<br>
>  endif()<br>
<br>
Generator expressions in install(FILES) are only allowed since CMake 3.0<br>
(<a href="https://cmake.org/cmake/help/v3.0/release/3.0.0.html#commands" rel="noreferrer" target="_blank">https://cmake.org/cmake/help/v3.0/release/3.0.0.html#commands</a>).<br>
The current minimum for libcxx is 2.8, so this should either be raised or we<br>
have to find another possibility of writing this install command...<br>
<br>
Greetings<br>
Jonas<br>
</blockquote></div>
</div></div></div>
<br></div></div><span>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
<br></span></blockquote></div><span><font color="#888888"><br><br clear="all"><div><br></div>-- <br><div><div dir="ltr">Alexey Samsonov<br><a href="mailto:vonosmas@gmail.com" target="_blank">vonosmas@gmail.com</a></div></div>
</font></span></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br><div><div dir="ltr">Alexey Samsonov<br><a href="mailto:vonosmas@gmail.com" target="_blank">vonosmas@gmail.com</a></div></div>
</font></span></div>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>