<div dir="ltr">Sorry about the breakage. I'm not sure what else to do.<div><br></div><div>The hack had been sitting there for more than a month IIRC.</div><div><br></div><div>Any advice for next time?</div><div><br></div><div>/Eric</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Feb 16, 2017 at 7:00 PM, Justin Bogner <span dir="ltr"><<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Eric Fiselier via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> writes:<br>
> Author: ericwf<br>
> Date: Sat Feb  4 19:19:02 2017<br>
> New Revision: 294116<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=294116&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project?rev=294116&view=rev</a><br>
> Log:<br>
> Remove CMake hack<br>
<br>
</span>FWIW, I just spent half an hour trying to figure out why a tree I hadn't<br>
built in a bit wouldn't build anymore, because it had LIBCXX_CXX_ABI=""<br>
in its cache. There isn't really a general "safe" timeframe to remove<br>
cmake variable upgrade hacks.<br>
<div class="HOEnZb"><div class="h5"><br>
> Modified:<br>
>     libcxx/trunk/CMakeLists.txt<br>
><br>
> Modified: libcxx/trunk/CMakeLists.txt<br>
> URL:<br>
> <a href="http://llvm.org/viewvc/llvm-project/libcxx/trunk/CMakeLists.txt?rev=294116&r1=294115&r2=294116&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project/libcxx/trunk/<wbr>CMakeLists.txt?rev=294116&r1=<wbr>294115&r2=294116&view=diff</a><br>
> ==============================<wbr>==============================<wbr>==================<br>
><br>
> --- libcxx/trunk/CMakeLists.txt (original)<br>
> +++ libcxx/trunk/CMakeLists.txt Sat Feb  4 19:19:02 2017<br>
> @@ -107,15 +107,6 @@ set(LIBCXX_CXX_ABI "default" CACHE STRIN<br>
>  set(CXXABIS none default libcxxabi libcxxrt libstdc++ libsupc++ vcruntime)<br>
>  set_property(CACHE LIBCXX_CXX_ABI PROPERTY STRINGS ;${CXXABIS})<br>
><br>
> -# FIXME: This is a temporary hack to force LLVM buildbots to store<br>
> -# the fixed cache entry instead of the previous cache entry. This is needed<br>
> -# because some LLVM buildbots don't clear their cache. It will be removed<br>
> -# once all LLVM bots have been run.<br>
> -if (LIBCXX_CXX_ABI STREQUAL "")<br>
> -  set(LIBCXX_CXX_ABI "default" CACHE STRING<br>
> -      "Specify the C++ ABI library to use." FORCE)<br>
> -endif()<br>
> -<br>
>  # Setup the default options if LIBCXX_CXX_ABI is not specified.<br>
>  if (LIBCXX_CXX_ABI STREQUAL "default")<br>
>    find_path(<br>
><br>
><br>
> ______________________________<wbr>_________________<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/<wbr>mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div>