The problem is you have to update gitignore every time you add a new test, and to make matters worse, sometimes there is already a Makefile there but the test suite generates a temporary one anyway, overwriting the in-tree one.<br><br>I think a real fix for this is going to involve generating all temporary files out of tree, or at least into one top level folder, and gitignoring that whole folder.  Each test could rm -rf the whole folder on startup, as it could just be a scratch area for tests<br><div class="gmail_quote">On Wed, Feb 4, 2015 at 7:52 AM Vince Harron <<a href="mailto:vharron@google.com">vharron@google.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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" target="_blank">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>
</blockquote></div>