<div dir="ltr">That list that lit checks for looks out-of-date and insufficient these days. Off the top of my head, I know for example that we need od, cd, rm, and mkdir, and there are probably several others that are necessary.<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 5 Oct 2021 at 15:51, via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">





<div style="overflow-wrap: break-word;" lang="EN-US">
<div class="gmail-m_5457158070550101787WordSection1">
<p class="MsoNormal">At a minimum, I think we should stop using (and recommending) the bit-rotting GnuWin32 tools, and instead use the tools that come with git-for-Windows.  Lit currently checks for 5 tools (cmp, diff, echo, grep, sed) and all of these are
 in the git-Windows set.  This will reduce the set of packages we depend on, even if it doesn’t reduce the set of actual tools we depend on.<u></u><u></u></p>
<p class="MsoNormal">--paulr<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<div style="border-color:currentcolor currentcolor currentcolor blue;border-style:none none none solid;border-width:medium medium medium 1.5pt;padding:0in 0in 0in 4pt">
<div>
<div style="border-color:rgb(225,225,225) currentcolor currentcolor;border-style:solid none none;border-width:1pt medium medium;padding:3pt 0in 0in">
<p class="MsoNormal"><b>From:</b> llvm-dev <<a href="mailto:llvm-dev-bounces@lists.llvm.org" target="_blank">llvm-dev-bounces@lists.llvm.org</a>> <b>On Behalf Of
</b>via llvm-dev<br>
<b>Sent:</b> Tuesday, October 5, 2021 10:30 AM<br>
<b>To:</b> <a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<b>Subject:</b> [llvm-dev] [llvm-lit] Old echo causes test failure on windows<u></u><u></u></p>
</div>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal"><span lang="EN-GB">Hi llvm-dev,<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">I wanted to bring discussion from D110986 to a wider audience.<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">After D110986 added a unicode character to the test, llvm\test\tools\llvm-cxxfilt\delimiters.test<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">started failing in our (Sony's) downstream Windows CI build (and locally for me) because the<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">unicode character was being converted to '?' at some stage in the test. The upstream Windows bots<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">seemed to have no such trouble. I added a workaround in D111072 that we landed because a Chromium<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">bot was running into the same issue.<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">Digging a little deeper it turns out that the echo that lit was picking up for me (`echo (GNU<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">coreutils) 5.3.0` from GnuWin32) converts unicode characters to '?'.<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">llvm-lit has its own built in echo which handles unicode without an issue. This is used in the workaround<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">D111072; lit's echo is used iff the echo command isn't used in a pipeline.<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">N.B. `echo (GNU coreutils) 8.32` shipped with git for Windows also appears to handle unicode without<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">an issue.<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">I have a few questions:<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">1) Is there a reason we cannot always use lit's echo, as Hans suggests in D111072?<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">2) If we can't always use lit's echo, maybe we could add an error/warning if a "bad" echo is<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">detected at build-config or test-running time?<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">Thanks,<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">Orlando<u></u><u></u></span></p>
</div>
</div>
</div>

_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>