<div dir="ltr">As long as the individual commands are still copy-pastable into both cmd and bash, I'm happy. I think practice this means the output should look more like this:<div><br></div><div>some test:</div><div>// RUN: %clang_cc1 %s -verify</div><div><br></div><div>output:</div><div>$ "C:\\src\\llvm\\build\\bin\\clang.exe" -cc1 "C:\\....\\test.cpp" -verify</div><div><br></div><div>Essentially every program used in the LLVM test suite understands forward slashes, so we could rewrite the path to test.cpp to use forward slashes, but I think cmd wants to see backslashes in argv0. Then our output would look like:</div><div>$ "C:\\src\\llvm\\build\\bin\\clang.exe" -cc1 C:/src/llvm/.../test.cpp -verify</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 31, 2016 at 4:53 PM, Daniel Dunbar via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">(+ the real cfe-dev)<div><div class="h5"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 31, 2016 at 4:48 PM, Daniel Dunbar <span dir="ltr"><<a href="mailto:daniel@zuster.org" target="_blank">daniel@zuster.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi all,<div><br></div><div>I have some patches up in:</div><div>  <a href="http://reviews.llvm.org/D20721" target="_blank">http://reviews.llvm.org/D20721</a></div><div>to make `lit` output more readable when commands fail. I'd like to get a little more feedback on this before landing, since it impacts how test failures appear when using the internal shell script runner (which is not the default so this will currently mostly impact Windows or test frameworks that intentionally set this behavior).</div><div><br></div><div>These patches change the output of a script like `true && echo hi && false` to something like:</div><div><br></div><div>```</div><div>$ true</div><div>$ echo hi</div><div>hi</div><div>    </div><div>$ false</div><div>note: command had no output on stdout or stderr</div><div>error: command failed with exit status 1</div><div>```</div><div><br></div><div>instead of the old:</div><div><br></div><div>```</div><div>Command 0: "true"</div><div>Command 0 Result: 0</div><div>Command 0 Output:</div><div><br></div><div><br></div><div>Command 0 Stderr:</div><div><br></div><div><br></div><div>Command 1: "echo" "hi"</div><div>Command 1 Result: 0</div><div>Command 1 Output:</div><div>hi</div><div><br></div><div><br></div><div>Command 1 Stderr:</div><div><br></div><div><br></div><div>Command 2: "wc" "missing-file"</div><div>Command 2 Result: 1</div><div>Command 2 Output:</div><div>None</div><div><br></div><div>Command 2 Stderr:</div><div>None</div><div>```</div><div><br></div><div>which I think is significantly more readable.</div><div><br></div><div>One change in the patches is that it will also start including, inline, the output of files which were redirected. This solves a common problem where a  command fails when it wasn't expected to, and its output was captured. In such cases the test log just showed the command failure, but since the output was redirected you couldn't tell from the log why the common failed.</div><div><br></div><div>Please let me know if you have any objections to the new format.</div><span><font color="#888888"><div><br></div><div> - Daniel</div><div><br></div></font></span></div>
</blockquote></div><br></div></div></div></div>
<br>_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
<br></blockquote></div><br></div>