-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/1137/#review2475
-----------------------------------------------------------



tests/SConscript
<http://reviews.gem5.org/r/1137/#comment2869>

    Can you access the use_colors variable from the top level sconscript? it 
has already figured this out.
    



tests/SConscript
<http://reviews.gem5.org/r/1137/#comment2870>

    Colors are already defined in m5.util.termcap so you can have 
termcap.Yellow or termcap.Blue


- Ali Saidi


On April 6, 2012, 9:17 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, 9:17 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
> -----
> 
>   tests/SConscript f51b4b4f0d5e 
> 
> 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

Reply via email to