[Lldb-commits] [PATCH] D12587: Roll dosep.py parallel test runner into dotest.py command line

[Todd Fiala via lldb-commits]

Yep, all good. I was planning on waiting for feedback first.

I've checked on Linux but don't have a Windows box available at the moment.

On Wednesday, September 2, 2015, Zachary Turner <zturner at google.com> wrote:

> zturner added a subscriber: zturner.
> zturner added a comment.
>
> 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.


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.

>
> 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)


Sure thing.

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.

>
>
> http://reviews.llvm.org/D12587
>
>
>
>

-- 
-Todd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20150902/7b156783/attachment.html>