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

[Todd Fiala via lldb-commits]

Please try:

./dotest.py --output-on-success -v --executable $INSTALLDIR/bin/lldb

I think you'll find that is identical output to what used to be this:

./dosep.py -s --options "-v --executable $INSTALLDIR/bin/lldb"

If you find differently, please let me know.  I couldn't maintain the '-s'
mapping for the dosep option because it collided with an existing dotest.py
option for session directory specification.

I'll wait to hear if that works for you.  (I tested it when I added it, and
I didn't mess with the logic aside from the flag name, so if it is broken I
definitely want to fix it --- you just have to use a different flag name).


On Fri, Sep 4, 2015 at 12:35 PM, <dawn at burble.org> wrote:

> On Fri, Sep 04, 2015 at 06:25:44PM +0000, Todd Fiala wrote:
> > tfiala added a comment.
> >
> > Dawn, the output supported by the command shouldn't have been changed.
> I'll have a peek at what may have caused that to happen.  All this change
> was trying to do was stop the need to call dosep and roll that behavior
> into dotest.
>
> Can you add the 'dosep.py -s -o "-v"' behavior to 'dotest.py -v'?  I
> think that may be all that's needed in order to show the same output as
> before with all the desired details.  Note: before the -s option was
> added to dosep.py, 'dosep.py -o "-v"' would only show the info that
> 'dotest.py -v' shows now (after your change).  Afer the -s option was
> added to dosep.py, 'dosep.py -s -o "-v"' would report the same info as
> 'dotest.py -v' did before your change.  I.e. dosep.py's -s option may be
> the key to fixing the regressions in 'dotest.py -v'.
>
> Thanks for looking into this!
> -Dawn
>



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