<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On Dec 21, 2013, at 4:16 PM, Alp Toker <<a href="mailto:alp@nuanti.com">alp@nuanti.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">To revisit this, I spent some time to build a database to avoid touching the source mtimes but it turned out to be a non-trivial amount of code.<br><br>It also lost one of the handy properties where we incidentally order on source modification time across multiple build output directories.<br><br>Another problem was "garbage collection" where if you run lit on a subdirectory, or when tests fail the database continues to grow and would need more logic for pruning.<br><br>Meanwhile I've had a lot of success on full dependency scanning which I'm going to complete and propose a few months into 2014. If that works out, it'll comprehensively replace this one so I don't think right now it's worth developing this approach beyond my original patch.<br><br>A few people have pinged on the patch so it's fair to say the patch is being applied and used locally.<br><br>For these reasons, I propose landing the original patch tweaked to be off by default and enabled with the short flag "-i" noting that source mtimes are modified in the help string.<br><br>Let me know if you think this is OK to land.<br></div></blockquote><div><br></div><div>It think there is enough utility/interest to justify adding an option since the proper solution will be much more work.</div><div>-Andy</div><br><blockquote type="cite"><div style="font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">Alp.<br><br><br><br>On 28/10/2013 02:04, Daniel Dunbar wrote:<br><blockquote type="cite">I like the functionality, but I will have to think some more about the implementation.<br><br>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.<br><br>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...<br><br>I'll think some more about this and get back to you with some more concrete ideas next week.<br><br>- Daniel<br><br>On Sunday, October 27, 2013, Andrew Trick wrote:<br><br> I really want this feature. It may not be clean, but would save a<br> lot of time in practice. However...<br><br> On Oct 24, 2013, at 10:35 PM, Alp Toker <<a href="mailto:alp@nuanti.com">alp@nuanti.com</a><br> <javascript:_e({}, 'cvml', '<a href="mailto:alp@nuanti.com">alp@nuanti.com</a>');>> wrote:<br><br><blockquote type="cite"> lit shell tests and compiled unit tests are both supported. At<br> the moment, the cache is implemented by scanning and modifying<br> test source mtimes directly. This hasn't caused trouble but the<br> technique could be refined in future to store test failures in a<br> separate cache file if needed.<br></blockquote><br> This doesn’t feel right. I don’t think it will personally cause me<br> any grief, but seems like someone or their tools will be confused.<br> If others think it’s ok, I won’t object. I just don’t want to<br> accept the patch until others review this aspect.<br><br> -Andy<br><br><br><br>--<span class="Apple-converted-space"> </span><br>- Daniel<br></blockquote><br>--<span class="Apple-converted-space"> </span><br><a href="http://www.nuanti.com/">http://www.nuanti.com</a><br>the browser experts</div></blockquote></div><br></body></html>