I like the functionality, but I will have to think some more about the implementation.<div><br></div><div>I have already considered having lit maintain some sort of database of test results. That would be useful for reporting things like flaky tests, reporting performance test changes, and for this feature.</div>
<div><br></div>The other thing that would be really nice to open the door for would be if lit could detect which tools have changed and only rerun tests for those tools. Implementing this feature by having lit keep a database of test results might provide a base for that...<div>
<br></div><div>I'll think some more about this and get back to you with some more concrete ideas next week.</div><div><br></div><div> - Daniel<br><div><br><div>On Sunday, October 27, 2013, Andrew Trick  wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word">I really want this feature. It may not be clean, but would save a lot of time in practice. However...<div><br><div><div>On Oct 24, 2013, at 10:35 PM, Alp Toker <<a href="javascript:_e({}, 'cvml', 'alp@nuanti.com');" target="_blank">alp@nuanti.com</a>> wrote:</div>
<br><blockquote type="cite"><span style="text-indent:0px;letter-spacing:normal;font-variant:normal;text-align:start;font-style:normal;display:inline!important;font-weight:normal;float:none;line-height:normal;text-transform:none;font-size:12px;white-space:normal;font-family:Helvetica;word-spacing:0px">lit shell tests and compiled unit tests are both supported. At the moment, the cache is implemented by scanning and modifying test source mtimes directly. This hasn't caused trouble but the technique could be refined in future to store test failures in a separate cache file if needed.</span><br style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px">
</blockquote></div><br></div><div>This doesn’t feel right. I don’t think it will personally cause me any grief, but seems like someone or their tools will be confused. If others think it’s ok, I won’t object. I just don’t want to accept the patch until others review this aspect.</div>
<div><br></div><div>-Andy</div></div></blockquote></div></div></div><br><br>-- <br> - Daniel<br>