----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1137/#review2486 -----------------------------------------------------------
I'm all for code that makes reading the regression output easier. Other possible ideas: 1. Supress non-pass/fail information so the relevant lines don't scroll right off the screen, forcing a rerun to get a summary (may already be possible?) 2. Sort passes and fails so fails are last and grouped together. 3. It's nice to see things pass/fail as they finish, so maybe let them finish normally while collecting results. Then sort them into passes and fails, and print a summary with passes in one group, fails in another, and maybe even a count of each. Then you can see if your super fast test failed right away without waiting for the slow one, and after all the regressions run all night, you don't have to run them again to actually see what happened. And failures won't be lost in the crowd. 4. Send the summary/sorted list in the emails. We've had failures we didn't see for a while because they were surrounded by a bunch of passes. - Gabe Black On April 6, 2012, 11:37 a.m., Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1137/ > ----------------------------------------------------------- > > (Updated April 6, 2012, 11:37 a.m.) > > > Review request for Default. > > > Description > ------- > > Regression: Add ANSI colours to highlight test status > > This patch adds a very basic pretty-printing of the test status > (passed or failed) to highlight failing tests even more: green for > passed, and red for failed. The printing only uses ANSI it the target > output is a tty and supports ANSI colours. Hence, any regression > scripts that are outputting to files or sending e-mails etc should > still be fine. > > > Diffs > ----- > > SConstruct 7a1dfb191e3f > tests/SConscript 7a1dfb191e3f > > Diff: http://reviews.gem5.org/r/1137/diff/ > > > Testing > ------- > > Ran util/regress in a terminal and also through regression > script > > > Thanks, > > Andreas Hansson > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
