<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Aug 21, 2014 at 2:21 PM, Brad King <span dir="ltr"><<a href="mailto:brad.king@kitware.com" target="_blank">brad.king@kitware.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="">On 08/21/2014 02:30 PM, Richard Shaw wrote:<br>

>    if   (wxWidgets_CXX_FLAGS)<br>
> +    # Flags are expected to be a string here, not a list.<br>
> +    string(REPLACE ";" " " wxWidgets_CXX_FLAGS "${wxWidgets_CXX_FLAGS}")<br>
>      set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${wxWidgets_CXX_FLAGS}")<br>
<br>
</div>Thanks.  However, we shouldn't leak our local modification to the value<br>
of wxWidgets_CXX_FLAGS, so the substitution should be done with the<br>
result stored in a temporary local variable.<br>
<br>
Also, please start the commit message in "UsewxWidgets: ...".<br><br></blockquote><div>Ok, all updates have been incorporated, so I'm looking for an ACK to know if it appears ready to push into "next".</div>
<div><br><a href="http://cmake.org/gitweb?p=stage/cmake.git;a=commitdiff;h=e6fa6e60f6330ddf60294a0d9a6ed4cb3f27d4c4">http://cmake.org/gitweb?p=stage/cmake.git;a=commitdiff;h=e6fa6e60f6330ddf60294a0d9a6ed4cb3f27d4c4</a></div>
<div><br></div><div>Thanks,</div><div>Richard </div></div><br></div></div>