<div><div>On Thu, Sep 30, 2010 at 11:04 AM, <span dir="ltr"><<a href="mailto:aaron.meadows@thomsonreuters.com">aaron.meadows@thomsonreuters.com</a>></span> wrote:</div></div><div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div lang="EN-US" link="blue" vlink="blue">
<div>
<p class="MsoNormal"><b><font size="2" color="gray" face="Arial"><span style="font-size:10.0pt;font-family:Arial;color:gray;font-weight:bold">Done. Issue
number 11279. [<a href="http://public.kitware.com/Bug/view.php?id=11279" target="_blank">http://public.kitware.com/Bug/view.php?id=11279</a>]</span></font></b></p>
<p class="MsoNormal"><b><font size="2" color="gray" face="Arial"><span style="font-size:10.0pt;font-family:Arial;color:gray;font-weight:bold"> </span></font></b></p>
<p class="MsoNormal"><b><font size="2" color="gray" face="Arial"><span style="font-size:10.0pt;font-family:Arial;color:gray;font-weight:bold">I also
noted that the script could be improved by calling the compiler with the ‘/?’
switch and parsing it’s output instead of parsing the output from
try_compile’ing a simple c file it writes. That would make it quite a
bit faster.</span></font></b></p></div></div></blockquote><div><br></div><div><div><div>Aaron,</div><div><br></div><div>Thanks for the patch. I've pushed it to git next.</div><div>Regarding your idea to use the output of "cl /?" to determine the version number, I think it is a good one. If you have time to rewrite the module to do this instead and test it I would merge the changes for a future version of CMake.</div>
<div><br></div><div>Some potential issues.</div><div>1. "cl /?" appears to pause waiting for input, this would have to be mitigated somehow in a safe way or perhaps just "cl" could be executed as that appears to include the output.</div>
<div>2. "cl" or "cl /?" output would have to be verified on VS8 to ensure it outputs there as well (it appears to be there in VS9/Express)</div><div>3. You couldn't just run "cl", you would have to find out the CMake variable for the compiler and run that.</div>
<div><br></div><div>My time is really limited. If you're interested, please create a new ticket and submit a patch. It would save probably at least a second off of an initial configure.</div></div></div><div><br></div>
</div>-- <br>Philip Lowman<br>
</div>