<div dir="rtl"><div dir="ltr">I removed the shell requirement r221271 so the test now is eanbled on Windows and</div><div dir="ltr">added a comment suggesting replacement of MSYS env.exe with the GnuWin32 one.</div><div dir="ltr"><br></div><div dir="ltr">MSYS2 also exhibits the root-replacing behaviour of env.exe.</div><div dir="ltr"><br></div><div dir="ltr"><br></div></div><div class="gmail_extra"><br><div class="gmail_quote"><div dir="ltr">2014-11-04 14:16 GMT+02:00 NAKAMURA Takumi <span dir="ltr"><<a href="mailto:geek4civic@gmail.com" target="_blank">geek4civic@gmail.com</a>></span>:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">It is working with gnuwin32's env.exe. I suggest you may introduce<br>
another feature to avoid msys toolchain.<br>
<br>
2014-08-10 6:24 GMT+09:00 Yaron Keren <<a href="mailto:yaron.keren@gmail.com">yaron.keren@gmail.com</a>>:<br>
> Author: yrnkrn<br>
> Date: Sat Aug  9 16:24:04 2014<br>
> New Revision: 215298<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=215298&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=215298&view=rev</a><br>
> Log:<br>
> Make this test unsupported when there is no real 'env' from a real shell.<br>
> While the test was already requiring shell-preserves-root (such as the<br>
> internal shell), it wasn't aware that the MSYS 1.0 env command knows how<br>
> to expand root by itself!<br>
><br>
> From cmd.exe try:<br>
><br>
>   env SDKROOT=/ cmd //c echo %SDKROOT%<br>
><br>
> And get:<br>
><br>
>   C:/MINGW/MSYS/1.0<br>
><br>
> To be certain we have a good 'env' program the test now requires a shell.<br>
><br>
><br>
> Modified:<br>
>     cfe/trunk/test/Driver/darwin-sdkroot.c<br>
><br>
> Modified: cfe/trunk/test/Driver/darwin-sdkroot.c<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/darwin-sdkroot.c?rev=215298&r1=215297&r2=215298&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/darwin-sdkroot.c?rev=215298&r1=215297&r2=215298&view=diff</a><br>
> ==============================================================================<br>
> --- cfe/trunk/test/Driver/darwin-sdkroot.c (original)<br>
> +++ cfe/trunk/test/Driver/darwin-sdkroot.c Sat Aug  9 16:24:04 2014<br>
> @@ -33,3 +33,9 @@<br>
>  //<br>
>  // It doesn't make sense on msys bash.<br>
>  // REQUIRES: shell-preserves-root<br>
> +// Even running under cmd.exe and not the msys shell, the msys 1.0 env<br>
> +// command will *not* preserve root, expanding / into C:/MINGW/MSYS/1.0.<br>
> +// To see, from the command prompt of cmd.exe run:<br>
> +//   env SDKROOT=/ cmd //c echo %SDKROOT%<br>
> +// To be sure we have real env program from a real shell and not the MSYS one:<br>
> +// REQUIRES: shell<br>
><br>
><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br></div>