<div dir="ltr">Hi Tzafrir,<br><br>This sounds like a good idea to me.<div><br></div><div><pre class="ndfHFb-c4YZDc-pf-DARUcf ndfHFb-c4YZDc-AHmuwe-wcotoc-th" style="font-family:'Courier New',Courier,monospace,arial,sans-serif;font-size:14px;margin-top:0px;margin-bottom:0px;white-space:pre-wrap;word-wrap:break-word;color:rgb(0,0,0);line-height:normal">+    subprocess.call(['rm','-rf',path+"*"])
</pre></div><div><br></div><div><br></div><div>This is super dangerous. Why the -f ?</div><div><br></div><div>Cheers,</div><div><br></div><div>James</div></div><br><div class="gmail_quote">On Thu, 14 May 2015 at 13:22 Tzafrir Poupko <<a href="mailto:tzafrir11@gmail.com">tzafrir11@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Before running a test, purge all %t files used by previous tests.<br><div class="gmail_quote"><div dir="ltr"><div>Remove any custom file used in test which was prefixed by %t.<br></div>There are over 300 clang tests that run "rm -rf %t" as first run command, this patch will make it automaticlly.<br></div>
</div><br></div>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>