<div dir="auto">It's needed: when clang-format does leave temp files behind, they will accumulate over builds and even if that's then fixed, the test would still fail because of the temp files from prior builds. Thankfully, takuni fixed this better in r296460.</div><div class="gmail_extra"><br><div class="gmail_quote">On Feb 28, 2017 4:14 AM, "Haojian Wu via cfe-commits" <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: hokein<br>
Date: Tue Feb 28 03:03:07 2017<br>
New Revision: 296453<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=296453&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project?rev=296453&view=rev</a><br>
Log:<br>
[clang-format] Fix test failure caused by "rm" on some buildbots.<br>
<br>
The begining command "rm" will return 1 when there is not such file to<br>
delete.<br>
<br>
This patch is to remove it, as it's not needed for the test.<br>
<br>
Modified:<br>
    cfe/trunk/test/Format/inplace.<wbr>cpp<br>
<br>
Modified: cfe/trunk/test/Format/inplace.<wbr>cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Format/inplace.cpp?rev=296453&r1=296452&r2=296453&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project/cfe/trunk/test/Format/<wbr>inplace.cpp?rev=296453&r1=<wbr>296452&r2=296453&view=diff</a><br>
==============================<wbr>==============================<wbr>==================<br>
--- cfe/trunk/test/Format/inplace.<wbr>cpp (original)<br>
+++ cfe/trunk/test/Format/inplace.<wbr>cpp Tue Feb 28 03:03:07 2017<br>
@@ -1,6 +1,5 @@<br>
 // Regression test to check that clang-format does not leave behind temporary<br>
 // files on Windows when doing in-place formatting.<br>
-// RUN: rm %T/inplace*<br>
 // RUN: cp %s %T/inplace.cpp<br>
 // RUN: clang-format -style=LLVM -i %T/inplace.cpp<br>
 // RUN: ls %T > %T/files.txt<br>
<br>
<br>
______________________________<wbr>_________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/cfe-commits</a><br>
</blockquote></div></div>