<div dir="ltr">It has been not maintained since <span style="font-size:13.1999998092651px;line-height:19.7999992370605px">r179283. and I haven't build mingw tree with autoconf. I don't object to drop shell-preservers-root. </span></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Jul 15, 2015 at 11:20 PM Yaron Keren <<a href="mailto:yaron.keren@gmail.com">yaron.keren@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="rtl"><div dir="ltr">Since r179283, internal shell is default on windows (*unless use_lit_shell = 0), so execute_external = not windows and</div><div dir="ltr"><br></div><div dir="ltr"><div dir="ltr"><font face="monospace, monospace"># Exclude MSYS due to transforming '/' to 'X:/mingwroot/'.</font></div><div dir="ltr"><font face="monospace, monospace">if not platform.system() in ['Windows'] or not execute_external:</font></div><div dir="ltr"><font face="monospace, monospace">    config.available_features.add('shell-preserves-root')</font></div><div><br></div><div>is really "if not windows or (not not windows)" = "if true" making <span style="font-family:monospace,monospace">shell-preserves-root</span> always true on Windows (unless use_lit_shell = 0).</div><div><br></div><div>No tests that require shell-preserves-root are actually disabled now on WIndows + MSYS2. In this configuration, just one test fails (which really require shell-preserves-root): darwin-sdkroot.c. This test has no purpose on windows anyhow and may be disabled. All other tests with shell-preserves-root do pass on MSYS2 bash.</div><div><br></div><div>Does shell-preserves-root serves any purpose now?</div><div>Any reason to keep it alive?</div><div><br></div></div></div>
</blockquote></div>