<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Remove rm invocations where the file is immediately rewritten later.<br>
<br>
This may or may not help making this test less flaky on windows. There's<br>
a race condition in lit somewhere.<br>
<br></blockquote><div><br></div><div>I've always assumed that this is just Windows holding onto the file for too long, possibly exacerbated by the presence of AV software, rather than an issue with lit itself although I'm just wildly speculating based on previous experience with this sort of thing.  I'm still hitting this failure on my machine occasionally today after this commit although I'll have to use it for a few days more before deciding whether it reduces the frequency of the failure or not.  </div><div><br></div><div>If it really is just a Windows thing rather than a lit thing I'm not sure what we'd be able to do beyond something like having lit intercept calls to rm and perform some sort of retry method for permission errors on Windows.  I'm pretty sure that will work, but it's extremely nasty, and if the problem is more heinous than just Windows holding onto files it's not a real fix either.</div><div><br></div><div>Thanks for having looked at this either way though!  It's a relief to know it's not just me that keeps running into this one.</div><div><br></div><div>-Greg</div><div><br></div><div><br></div><div><br></div><div><br></div><div><br></div></div></div></div>