<p dir="ltr">The tests do clean up after themselves, unless they're interrupted.  This is to prevent temporary files that can't be moved out of tree from showing up in git status.</p>
<div class="gmail_quote">On Feb 4, 2015 7:49 AM, "Zachary Turner" <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+1 for not touching gitignore.  Seems like a bandaid for the real problem.  Tests should be smart enough to clean up after themselves.  <br><br>I thought there were cases where a Makefile was checked into tree.  This would also delete those right?<br><div class="gmail_quote">On Wed, Feb 4, 2015 at 7:22 AM Pavel Labath <<a href="mailto:labath@google.com" target="_blank">labath@google.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I'm particularly fond of that idea. Even if I put full paths to all 10 makefiles to .gitignore now (and I need to put full paths, as we don't want to ignore all makefiles), the list will get quickly out of sync as test cases are added/removed.<br>
<br>
Besides, if this patch goes in in this form, adding these makefiles back will be pretty difficult as they will get deleted after every testsuite run.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D7407" target="_blank">http://reviews.llvm.org/D7407</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/<u></u>settings/panel/<u></u>emailpreferences/</a><br>
<br>
<br>
<br>
______________________________<u></u>_________________<br>
lldb-commits mailing list<br>
<a href="mailto:lldb-commits@cs.uiuc.edu" target="_blank">lldb-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/lldb-commits</a><br>
</blockquote></div>
</blockquote></div>