Looks good to me. (Visual inspection only...)<div><br></div><div>One thing I noticed about the topic branch, though. You've got two email addresses listed as commit authors, but they're both you... Did you want to make those consistent before they go into CMake?<br>
<br><br><div class="gmail_quote">On Tue, Aug 21, 2012 at 5:18 AM, Stephen Kelly <span dir="ltr"><<a href="mailto:steveire@gmail.com" target="_blank">steveire@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Hi,<br>
<br>
I've pushed the qt5-qtdialog-port branch to stage. It makes it possible to<br>
build the QtDialog with Qt 5 (and some other clean-ups). I don't have Qt 4<br>
in my Qt 5 environment, and I do need cmake in my Qt 5 environment, so this<br>
ensures that I get a usable dialog when building cmake in my Qt 5<br>
environment.<br>
<br>
In the branch, Qt 5 is preferred if found.<br>
<br>
Any objection to merging it to next?<br>
<br>
Thanks,<br>
<br>
Steve.<br>
<br>
<br>
--<br>
<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Please keep messages on-topic and check the CMake FAQ at: <a href="http://www.cmake.org/Wiki/CMake_FAQ" target="_blank">http://www.cmake.org/Wiki/CMake_FAQ</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers" target="_blank">http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers</a><br>
</blockquote></div><br></div>