| Summary: | Add an option to run-webkit-tests to print stderr/stdout from the tool | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Simon Fraser (smfr) <simon.fraser> | ||||
| Component: | Tools / Tests | Assignee: | David Farler <dfarler> | ||||
| Status: | NEW --- | ||||||
| Severity: | Normal | CC: | ap, commit-queue, dbates, ddkilzer, dfarler, glenn, jake.nielsen.webkit | ||||
| Priority: | P2 | ||||||
| Version: | 528+ (Nightly build) | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=168468 | ||||||
| Attachments: |
|
||||||
|
Description
Simon Fraser (smfr)
2014-10-10 17:00:21 PDT
Created attachment 239663 [details]
Patch
--debug-rwt-logging already prints stderr. I'm not quite sure what the use case for the new flag is, but maybe it's best to have it only control stdout printing, and keep stderr for the existing one? --debug-rwt-logging is extremely noisy and is for debugging webkitpy code. This option is just to have the results transparently show up without having to go to the result files (I assume that's what smfr is requesting). Makes sense, although it's also fairly easy to just run the tool without run-webkit-tests wrapper to get these. +1 for adding the option. Comment on attachment 239663 [details]
Patch
This seems like a useful change. r=me.
|