RESOLVED FIXED 93026
nrwt: clean up logging, pt. 2: clean up printing.py 
https://bugs.webkit.org/show_bug.cgi?id=93026
Summary nrwt: clean up logging, pt. 2: clean up printing.py
Dirk Pranke
Reported 2012-08-02 13:56:47 PDT
nrwt: clean up printing.py
Attachments
Patch (39.73 KB, patch)
2012-08-02 13:58 PDT, Dirk Pranke 		no flags
DetailsFormatted DiffDiff
Patch for landing (45.14 KB, patch)
2012-08-02 17:41 PDT, Dirk Pranke 		dpranke: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Dirk Pranke
Comment 1 2012-08-02 13:58:27 PDT
Created attachment 156159 [details] Patch
Ojan Vafai
Comment 2 2012-08-02 14:09:29 PDT
Comment on attachment 156159 [details] Patch nice!
Dirk Pranke
Comment 3 2012-08-02 17:41:07 PDT
Created attachment 156217 [details] Patch for landing
Dirk Pranke
Comment 4 2012-08-02 17:41:44 PDT
Comment on attachment 156217 [details] Patch for landing this fixes some grammatical nits and spacing as well as a bug with printing unexpected results always.
Dirk Pranke
Comment 5 2012-08-06 13:46:46 PDT
Committed r124800: <http://trac.webkit.org/changeset/124800>
Thiago Marcos P. Santos
Comment 6 2012-08-07 01:50:41 PDT
After this change, instead of the bots displaying some useful information like: Aug 06 13:53 124800 failure EFL Linux 64-bit Debug #3930 Failed 9 failures 7 flakes It is now showing: Aug 06 13:53 124800 failure EFL Linux 64-bit Debug #3930 Failed layout-test
Csaba Osztrogonác
Comment 7 2012-08-07 02:00:05 PDT
(In reply to comment #6) > After this change, instead of the bots displaying some useful information like: > > Aug 06 13:53 124800 failure EFL Linux 64-bit Debug #3930 Failed 9 failures 7 flakes > > It is now showing: > > Aug 06 13:53 124800 failure EFL Linux 64-bit Debug #3930 Failed layout-test New bug report: https://bugs.webkit.org/show_bug.cgi?id=93346 Patch is coming soon.
Dirk Pranke
Comment 8 2012-08-07 15:29:05 PDT
I thought I had checked this before making the change, but I guess not. Thanks for taking care of it!
Note You need to log in before you can comment on or make changes to this bug.