<div dir="ltr"><div>This is not important, but when I build libcxx using a recent beta version cmake, </div><div>namely this one "cmake version 2.8.11.20130809-gba97e"</div><div> </div><div>I get this error now:</div>
<div> </div><div>CMake Warning (dev) in CMakeLists.txt:<br>  Syntax Warning in cmake code at</div><p>    c:/libcxx/CMakeLists.txt:48:54</p><div>  Argument not separated from preceding token by whitespace.<br>This warning is for project developers.  Use -Wno-dev to suppress it.</div>
<div> </div><div>The offending line 48 appears to be this:</div><div> </div><div>set_property(CACHE LIBCXX_CXX_ABI PROPERTY STRINGS "";${CXXABIS})</div><div> </div><div>Removing the quotes and changing the line to the line below fixes the warning and I hope does not change the meaning?</div>
<div>I've yet to observe any harm from the change on my build at least.</div><div> </div><div>set_property(CACHE LIBCXX_CXX_ABI PROPERTY STRINGS ;${CXXABIS})</div><div> </div><div>This is all the attached patch does.</div>
<div> </div><div>The ; is curious as to why it's needed but I do remember a problem vaguely that I fixed in this file some time ago related to ; delimiters and maybe CXXABIS that suggests it is necessary so I've left the ;</div>
<div> </div><div>I mention that in case it allows anyone to recall the conversation that will allow them to remove the ";" too. But otherwise they probably shouldn't.</div><div>As long as "";$(X) yields the same string as ;$(x) it should be a safe change. Hopefully a regular cmake user can verify this.</div>
<div> </div><div>It's not a necessary fix it just removes an irritant warning that can be supressed with a flag anyway.<br><br><br></div></div>