<br><br><div class="gmail_quote">2009/8/13 Daniel Dunbar <span dir="ltr"><<a href="mailto:daniel@zuster.org">daniel@zuster.org</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Hi Baptiste,<br>
<br>
This patch isn't quite correct, the situation is both more complicated<br>
and simpler. What the script needs to do is just model the file<br>
mappings (which start out as the default), and then apply each<br>
redirection to that model. Then at the end, it needs to verify that we<br>
can call the subprocess module with the right settings to match the<br>
redirection.<br>
<br>
For now, a simpler patch which I would be fine with is to just change<br>
those 4 tests to use a temporary file instead of shell redirection<br>
fun, e.g.:<br>
<br>
RUN: clang 2> %t<br>
RUN: grep 'file ...' %t | count 3<br>
<font color="#888888"><br>
 - Daniel</font></blockquote><div><br>You're right, this will work just as well. It is a lot simpler and remove the /dev/null issue.<br><br>Can you elaborate why the patch is incorrect? From the tests I made the stderr of clang-cc is correctly redirected to stdin of count/grep... What am I missing?<br>
 </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><font color="#888888"></font>On Tue, Aug 11, 2009 at 3:19 PM, Baptiste<br><div class="im">

Lepilleur<<a href="mailto:baptiste.lepilleur@gmail.com">baptiste.lepilleur@gmail.com</a>> wrote:<br>
</div><div><div></div><div class="h5">> This patch fixes failures of test such as:<br>
> // RUN: clang-cc -E %s -fno-caret-diagnostics 2>&1 >/dev/null | grep 'file<br>
> successfully included' | count 3<br>
><br>
> when run on Windows with the shell emulation.</div></div></blockquote><div> </div></div>