<div class="gmail_quote">On Mon, Jun 4, 2012 at 4:41 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 06/04/2012 10:25 AM, Bill Hoffman wrote:<br>
> On 6/1/2012 1:55 PM, Brad King wrote:<br>
>> The generator is supposed to produce a file that lists all the<br>
>> compiler invocation command lines in a JSON format.  The test<br>
>> runs a build with the option and then runs a program that reads<br>
>> the generated file and re-plays the commands.  The test and its<br>
>> C++ program should not need modification because it works for the<br>
>> Makefile generator.  The Ninja generator will need to generate<br>
>> the properly encoded commands like the Makefile generator does.<br>
><br>
> Seems like the json file is reasonable:<br>
><br>
</div><div class="im">>    "command": "C:\\PROGRA~2\\MICROS~1.0\\VC\\bin\\cl.exe   /nologo<br>
> /DWIN32 /D_WINDOWS /W3 /Zm1000 /EHsc /GR /D_DEBUG /MDd /Zi /Ob0 /Od<br>
> /RTC1 -I\"c:\\Users\\hoffman\\Work\\My<br>
> Builds\\NinjaCMake\\Tests\\CompileCommandOutput\\..\\..\\Source\"<br>
> /TP<br>
> /FoCMakeFiles/CompileCommandOutput.dir/compile_command_output.cxx.obj<br>
> /FdTARGET_PDB -c \"c:\\Users\\hoffman\\Work\\My<br>
> Builds\\NinjaCMake\\Tests\\CompileCommandOutput\\compile_command_output.cxx\"",<br>
>    "file": "c:\\Users\\hoffman\\Work\\My<br>
> Builds\\NinjaCMake\\Tests\\CompileCommandOutput\\compile_command_output.cxx"<br>
> },<br>
><br>
</div><div class="im">> the test driver should be able to read a file with \\ in it,<br>
> and the generator output is fine.<br>
<br>
</div>The topic was first brought up here:<br>
<br>
 <a href="http://thread.gmane.org/gmane.comp.programming.tools.cmake.devel/3678" target="_blank">http://thread.gmane.org/gmane.comp.programming.tools.cmake.devel/3678</a><br>
<br>
Context of this discussion:<br>
<br>
 <a href="http://thread.gmane.org/gmane.comp.programming.tools.cmake.devel/3774" target="_blank">http://thread.gmane.org/gmane.comp.programming.tools.cmake.devel/3774</a><br>
<br>
Origin of parser in question:<br>
<br>
 <a href="http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=0e6b05fc" target="_blank">http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=0e6b05fc</a><br>
<br>
Manuel, as author of this parser can you comment please?</blockquote><div><br></div><div>From taking a very high level look it seems to me like the parser probably needs fixing on Windows - I unfortunately don't have a Windows machine around (I really need to get one ...), but I think disabling it for Windows is fine for now - as Stephen points out in a later email clang tooling for Windows is not that useful yet due to clang restrictions. If somebody expresses a strong need for the Windows fixes, I can prioritize them; please let me know in that case.</div>
<div><br></div><div>Cheers,</div><div>/Manuel</div></div>