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

Todd Fiala via lldb-commits lldb-commits at lists.llvm.org
Wed Sep 2 23:23:41 PDT 2015

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

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

More information about the lldb-commits mailing list