<div dir="ltr"><div style>I happen to have Cygwin bash on my PATH, so shell is true, but I've found that manually setting 'execute_external = False' in lit.cfg lets me run far more of the test suite.  Perhaps I should remove the shell feature in that case.</div>
<div><br></div>However, I don't think this test can work at all on Windows, unless TestRunner.py tries harder to replace /dev/null.  The command line is 'opt -o /dev/null /dev/null 2>&-', and opt knows nothing about /dev/null, right?  For me that file does not exist and I get a non-zero return code for the second opt invocation.</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jan 25, 2013 at 6:47 PM, NAKAMURA Takumi <span dir="ltr"><<a href="mailto:geek4civic@gmail.com" target="_blank">geek4civic@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Reid, I suppose it has been disabled with "REQUIRES:shell" on win32<br>
(includes msvc).<br>
What is your motivation on this commit?<br>
<br>
FYI, it has been working with mingw32 msys bash.<br>
<br>
...Takumi<br>
<br>
2013/1/26 Reid Kleckner <<a href="mailto:reid@kleckner.net">reid@kleckner.net</a>>:<br>
<div class="HOEnZb"><div class="h5">> Author: rnk<br>
> Date: Fri Jan 25 16:12:54 2013<br>
> New Revision: 173509<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=173509&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=173509&view=rev</a><br>
> Log:<br>
> XFAIL close-stderr on win32<br>
><br>
> The test runner does not rewrite instances of /dev/null inside the<br>
> quoted sh command.  /dev/null does not exist, so opt will fail to open<br>
> it, and return a non-zero exit code.<br>
><br>
> Modified:<br>
>     llvm/trunk/test/Other/close-stderr.ll<br>
><br>
> Modified: llvm/trunk/test/Other/close-stderr.ll<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Other/close-stderr.ll?rev=173509&r1=173508&r2=173509&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Other/close-stderr.ll?rev=173509&r1=173508&r2=173509&view=diff</a><br>

> ==============================================================================<br>
> --- llvm/trunk/test/Other/close-stderr.ll (original)<br>
> +++ llvm/trunk/test/Other/close-stderr.ll Fri Jan 25 16:12:54 2013<br>
> @@ -9,5 +9,8 @@<br>
>  ; XFAIL: vg_leak<br>
>  ; REQUIRES: shell<br>
><br>
> +; opt will fail to open /dev/null on native win32.<br>
> +; XFAIL: win32<br>
> +<br>
>  ; Test that the error handling when writing to stderr fails exits the<br>
>  ; program cleanly rather than aborting.<br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>