<div dir="ltr"><div class="gmail_quote"><div class="gmail_attr">Hi Paul,<br></div><div dir="ltr" class="gmail_attr"><br></div><div dir="ltr" class="gmail_attr">On Fri, Jul 26, 2019 at 11:56 AM <<a href="mailto:paul.robinson@sony.com">paul.robinson@sony.com</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">> We have had an internal bot get 9 failures in the lit test suite<br>
> since this patch went in; I will see if the suggested patches help.<br>
> --paulr<br>
<br>
They don't help.  It appears the gnuwin32 'env' does understand -u so<br>
that wasn't the problem.<br></blockquote><div><br></div><div>Bummer.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Picking apart 'googletest-format.py' and doing the steps manually,<br>
so that I can get FileCheck to tell me what's going on, it appears<br>
that the .out file generated by the test has an extra line terminator,<br>
so a CHECK-NEXT on the .out file fails.<br></blockquote><div><br></div><div>Where does that line terminator appear?<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
If I hack the lit.cfg so it's just 'env %{python} etc' (not trying<br>
to set or unset) the same failure occurs; so, something about running<br>
python or lit.py under GnuWin32 'env' is making this occur.<br>
<br>
env version 5.3.0<br>
python version 2.7.12 on Windows version 7<br>
python version 2.7.14 on Windows version 10<br></blockquote><div><br></div><div>If you're pursuing this, let me know.  Otherwise, I'll move to windows and try to figure this out.</div><div><br></div><div>Thanks for the details.<br></div><div><br></div><div>Joel<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
--paulr<br>
<br>
> <br>
> > -----Original Message-----<br>
> > From: Michał Górny [mailto:<a href="mailto:mgorny@gentoo.org" target="_blank">mgorny@gentoo.org</a>]<br>
> > Sent: Friday, July 26, 2019 10:28 AM<br>
> > To: Joel E. Denny;<br>
> <a href="mailto:reviews%2BD65156%2Bpublic%2B5784c7837e421597@reviews.llvm.org" target="_blank">reviews+D65156+public+5784c7837e421597@reviews.llvm.org</a><br>
> > Cc: Robinson, Paul; Dan Liew; Thomas Preud'homme; LLVM Commits<br>
> > Subject: Re: [PATCH] D65156: [lit] Protect full test suite from<br>
> > FILECHECK_OPTS<br>
> ><br>
> > On Fri, 2019-07-26 at 10:17 -0400, Joel E. Denny wrote:<br>
> > > Hi Michał,<br>
> > ><br>
> > > On Fri, Jul 26, 2019 at 10:12 AM Michał Górny via Phabricator <<br>
> > > <a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>> wrote:<br>
> > ><br>
> > > > mgorny added a comment.<br>
> > > ><br>
> > > > This change broke NetBSD buildbots, e.g.:<br>
> > > > <a href="http://lab.llvm.org:8011/builders/netbsd-" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/netbsd-</a><br>
> ><br>
> amd64/builds/21152/steps/run%20unit%20tests/logs/FAIL%3A%20lit%3A%3A%20goo<br>
> > gletest-upstream-format.py<br>
> > ><br>
> > > I didn't receive an automated email for this.  Should I have?<br>
> > ><br>
> ><br>
> > To be honest, I don't really know.  The bot hasn't managed to get green<br>
> > for quite some time already, and I don't know if it pings you if there<br>
> > is a constant breakage.<br>
> ><br>
> > --<br>
> > Best regards,<br>
> > Michał Górny<br>
<br>
</blockquote></div></div>