<div dir="ltr">They make the test suite run faster. when a file wasn't changed.</div><br><div class="gmail_quote"><div dir="ltr">On Mon, Apr 18, 2016 at 11:34 AM Jim Ingham <<a href="mailto:jingham@apple.com">jingham@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">That I don't know.  But we really shouldn't be generating the .pyc files, they just litter the test directories and I don't think they are much help.<br>
<br>
Jim<br>
<br>
> On Apr 18, 2016, at 11:29 AM, Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>> wrote:<br>
><br>
> That's normal, but why would that cause an issue for the test suite?  What is leading to it thinking that it should execute a .pyc file in the first place?<br>
><br>
> On Mon, Apr 18, 2016 at 11:25 AM Jim Ingham <<a href="mailto:jingham@apple.com" target="_blank">jingham@apple.com</a>> wrote:<br>
> IIRC, python will generate byte-compiled ".pyc" versions of the ".py" files automatically (and helpfully leave them in the CWD...)  We turned that off for the testsuite through some Python magic I don't think I ever knew, but if I did I've certainly forgotten it...  Anyway, sounds like that got turned back on again, and we should turn it off again...<br>
><br>
> Jim<br>
><br>
> > On Apr 18, 2016, at 11:01 AM, Zachary Turner via lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a>> wrote:<br>
> ><br>
> > It seems very strange to me to be changing a .pyc filename to a .py filename.  I think we should try to understand where the .pyc filename is coming from to begin with, and this is just masking the real error.<br>
> ><br>
> > On Mon, Apr 18, 2016 at 10:10 AM Pavel Labath via lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a>> wrote:<br>
> > labath accepted this revision.<br>
> > labath added a comment.<br>
> > This revision is now accepted and ready to land.<br>
> ><br>
> > Ok, makes sense then, thanks for making sure it's necessary.<br>
> ><br>
> > I don't quite understand how the string makes it's way there though, as the only place setting this field in the entire repository seems to be lldbinline.py<br>
> ><br>
> ><br>
> > <a href="http://reviews.llvm.org/D19215" rel="noreferrer" target="_blank">http://reviews.llvm.org/D19215</a><br>
> ><br>
> ><br>
> ><br>
> > _______________________________________________<br>
> > lldb-commits mailing list<br>
> > <a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a><br>
> > <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits</a><br>
> > _______________________________________________<br>
> > lldb-commits mailing list<br>
> > <a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a><br>
> > <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits</a><br>
><br>
<br>
</blockquote></div>