[Lldb-commits] [Diffusion] rL246794: Roll dosep.py parallel test runner into dotest.py command line
Todd Fiala via lldb-commits
lldb-commits at lists.llvm.org
Fri Sep 4 12:44:36 PDT 2015
./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!
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the lldb-commits