Re: [Lldb-commits] [PATCH] D12416: Parse dotest.py/dosep.py output, providing general stats and skip reason breakdown by counts.

2015-10-10 Thread Todd Fiala via lldb-commits
tfiala abandoned this revision. tfiala added a comment. I'm going to redo this some time in the future based on the test event architecture. http://reviews.llvm.org/D12416 ___ lldb-commits mailing list lldb-commits@lists.llvm.org

Re: [Lldb-commits] [PATCH] D12416: Parse dotest.py/dosep.py output, providing general stats and skip reason breakdown by counts.

2015-08-31 Thread Greg Clayton via lldb-commits
> On Aug 28, 2015, at 9:36 AM, Zachary Turner wrote: > > > > On Fri, Aug 28, 2015 at 8:41 AM Todd Fiala wrote: > tfiala added a comment. > > In http://reviews.llvm.org/D12416#235112, @labath wrote: > > > Same question as Zachary. This sounds like a

Re: [Lldb-commits] [PATCH] D12416: Parse dotest.py/dosep.py output, providing general stats and skip reason breakdown by counts.

2015-08-28 Thread Pavel Labath via lldb-commits
labath added a comment. In http://reviews.llvm.org/D12416#235166, @tfiala wrote: instead of making another layer on top of that (it's bad enough we have dosep and dotest already). The existence of those two is at least partly due to the organic nature in which they developed. I

Re: [Lldb-commits] [PATCH] D12416: Parse dotest.py/dosep.py output, providing general stats and skip reason breakdown by counts.

2015-08-28 Thread Greg Clayton via lldb-commits
clayborg requested changes to this revision. clayborg added a comment. This revision now requires changes to proceed. I would like to see a few things as long as we are chaning things: - get rid of dosep.py and just put the functionality into dotest.py if possible, it can just spawn itself in

Re: [Lldb-commits] [PATCH] D12416: Parse dotest.py/dosep.py output, providing general stats and skip reason breakdown by counts.

2015-08-28 Thread Zachary Turner via lldb-commits
On Fri, Aug 28, 2015 at 8:41 AM Todd Fiala todd.fi...@gmail.com wrote: tfiala added a comment. In http://reviews.llvm.org/D12416#235112, @labath wrote: Same question as Zachary. This sounds like a very useful feature and it would be nice to have it integrated into the current test system,

Re: [Lldb-commits] [PATCH] D12416: Parse dotest.py/dosep.py output, providing general stats and skip reason breakdown by counts.

2015-08-27 Thread Todd Fiala via lldb-commits
tfiala added a comment. Note the output is formatted for whitespace and left justified, with the numeric columns all lined up. My verbatim mode above doesn't look like it did what I expected... http://reviews.llvm.org/D12416 ___ lldb-commits

Re: [Lldb-commits] [PATCH] D12416: Parse dotest.py/dosep.py output, providing general stats and skip reason breakdown by counts.

2015-08-27 Thread Todd Fiala via lldb-commits
tfiala updated this revision to Diff 33375. http://reviews.llvm.org/D12416 Files: test/reports/dotest_stats.py Index: test/reports/dotest_stats.py === --- test/reports/dotest_stats.py +++ test/reports/dotest_stats.py @@ -0,0

[Lldb-commits] [PATCH] D12416: Parse dotest.py/dosep.py output, providing general stats and skip reason breakdown by counts.

2015-08-27 Thread Todd Fiala via lldb-commits
tfiala created this revision. tfiala added a reviewer: clayborg. tfiala added a subscriber: lldb-commits. The new script can be run against output of dotest.py/dosep.py, with the primary benefit of providing a detailed breakdown of the number of test methods that skip for a given reason.

Re: [Lldb-commits] [PATCH] D12416: Parse dotest.py/dosep.py output, providing general stats and skip reason breakdown by counts.

2015-08-27 Thread Todd Fiala via lldb-commits
tfiala added a comment. And, the file is actually in test/reports/, not reports/... http://reviews.llvm.org/D12416 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits