Yep, all good. I was planning on waiting for feedback first.<div><br></div><div>I've checked on Linux but don't have a Windows box available at the moment.<br><div><br>On Wednesday, September 2, 2015, Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">zturner added a subscriber: zturner.<br>
zturner added a comment.<br>
<br>
Are there any plans to actually just kill dosep.py at some point?  I know you mentioned you want to do further cleanup, so I'm wondering if this is just an incremental step towards a dosep'less world.</blockquote><div><br></div><div>Right. At this point dosep could have been hidden in a lib dir or just wholesale pulled into dotest.py, but it is now an implementation detail that can be refactored. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I'll review this in more detail tomorrow, but please hold off on comitting until I get a chance to test everything on Windows and make sure everything still works.  (I assume other people will want to do the same for their platforms)</blockquote><div><br></div><div>Sure thing.</div><div><br></div><div>The implementation I went with was minimal change to get the net result of the user experience being "just dotest.py", with concurrent test execution the default. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<a href="http://reviews.llvm.org/D12587" target="_blank">http://reviews.llvm.org/D12587</a><br>
<br>
<br>
<br>
</blockquote></div></div><br><br>-- <br><div dir="ltr">-Todd</div><br>