Closed
Bug 661940
Opened 14 years ago
Closed 14 years ago
Allow turning on verbose reftest output without modifying reftest.js
Categories
(Testing :: Reftest, enhancement)
Testing
Reftest
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla7
People
(Reporter: cjones, Assigned: cjones)
Details
Attachments
(1 file, 1 obsolete file)
|
2.26 KB,
patch
|
roc
:
review+
|
Details | Diff | Splinter Review |
No description provided.
|
Assignee | |
Comment 1•14 years ago
|
||
Attachment #537235 -
Flags: review?(roc)
Seems like you should suppress the dumping of gTestLog when gVerbose is set, right? We don't need to see the same set of messages twice.
|
|
Assignee | |
Comment 3•14 years ago
|
||
I don't think a human would care, and it might be useful for a script to be able to parse out only log snippets from failed tests. But I can suppress it if you'd prefer.
I think it might be confusing to people reading the log.
|
|
Assignee | |
Comment 5•14 years ago
|
||
OK.
Attachment #537235 -
Attachment is obsolete: true
Attachment #537235 -
Flags: review?(roc)
Attachment #537280 -
Flags: review?(roc)
Comment on attachment 537280 [details] [diff] [review]
Dump verbose messages if MOZ_REFTEST_VERBOSE is set in the environment, v2
Review of attachment 537280 [details] [diff] [review]:
-----------------------------------------------------------------
Attachment #537280 -
Flags: review?(roc) → review+
|
|
Assignee | |
Comment 7•14 years ago
|
||
Whiteboard: [inbound]
|
||
Comment 8•14 years ago
|
||
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla7
You need to log in
before you can comment on or make changes to this bug.
Description
•