<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Hi,<div><br><div><div>Am 14.07.2012 um 22:08 schrieb David Cole:</div><br class="Apple-interchange-newline"><blockquote type="cite">On Sat, Jul 14, 2012 at 3:30 PM, Patrick Gansterer <span dir="ltr"><<a href="mailto:paroga@paroga.com" target="_blank">paroga@paroga.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hi,<div><br>
<br>
On Sat, 14 Jul 2012 21:08:24 +0200, Rolf Eike Beer wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
-in Modules/Platform/Windows-cl.<u></u>cmake you initialize some variables in both<br>
branches of an if(). You should keep the order you set them the same in both<br>
branches as is makes comparing this much easier.<br>
</blockquote>
<br></div>
Where exactly? I don't find it. :-/<div><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
-you don't set CMAKE_DEBUG_PLATFORM_DEFINES for WinCE. I don't know<br>
if that is<br>
a bug or intentional. If it is a bug then set it, otherwise I would add a<br>
comment why this isn't needed.<br>
</blockquote>
<br></div>
The WinCE compier has no support for /RTC1.<br>
<br>
This patch also includes the changes mentioned in <a href="http://public.kitware.com/pipermail/cmake-developers/2011-November/002510.html" target="_blank">http://public.kitware.com/<u></u>pipermail/cmake-developers/<u></u>2011-November/002510.html</a>.<span><font color="#888888"><br>


<br>
-- Patrick</font></span><div><div><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/<u></u>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/<u></u>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-<u></u>bin/mailman/listinfo/cmake-<u></u>developers</a><br>
</div></div></blockquote></div><br><div><br></div><div>Patrick,</div><div><br></div><div>Thanks for your work on this. I'll review this stuff on Monday or Tuesday and apply the patches and merge to next if all looks well.</div>

<div><br></div><div>Part of the problem I've always had with "other platform" support on Windows or Visual Studio is that we have literally ZERO dashboard coverage of these platforms.</div><div><br></div><div>

Is there any way you (or somebody else who cares about these platforms) can set up a CMake dashboard to test the platform and to make sure that it submits nightly so that if we change something surrounding this code, we have a chance at catching unintended breakages *before* we make a CMake release?</div></blockquote><div><br></div><div>I'll have a look at it and try to set something up.</div><br><blockquote type="cite">

<div><br></div><div>This thread, or one like it, probably deserves a cc to the users list as well to see if somebody there can submit such dashboards. From the activity in mantis, it does seem to be a popular issue. Lots of folks will be glad to see stuff like this go in.</div>
</blockquote><br></div><div>-- Patrick</div><br></div></body></html>