Bug 60324 - NRWT: remove --print detailed-progress
Summary: NRWT: remove --print detailed-progress
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Dirk Pranke
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-05 17:18 PDT by Dirk Pranke
Modified: 2011-05-27 19:12 PDT (History)
7 users (show)

See Also:

Attachments
Patch (17.53 KB, patch)
2011-05-05 17:19 PDT, Dirk Pranke 		abarth: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Pranke 2011-05-05 17:18:10 PDT 
NRWT: remove --print detailed-progress
Comment 1 Dirk Pranke 2011-05-05 17:19:34 PDT 
Created attachment 92508 [details]
Patch
Comment 2 Dirk Pranke 2011-05-05 17:23:26 PDT 
At some point (more than a year ago), I added a "--print detailed-progress" option to new-run-webkit-tests to try and emulate the output that old-run-webkit-tests did (printing the directory name followed by a dot at a time for each test).

The code was always something of a hack since the default mode (running tests in parallel sharded by directory) can't really be mapped onto that style of output. So, the detailed progress only worked if you weren't running things in parallel at all, or if you were running in "experimental-fully-parallel" mode where we put each test into a shard and did roughly try to run them in order.

There was also a fair amount of complexity to handle the tests coming in that way and printing things out.

Given the year since, I don't think I've used this feature a single time, and I haven't heard of anyone else using it, either. 

So, I'd like to remove it.
Comment 3 Adam Barth 2011-05-05 17:43:28 PDT 
Comment on attachment 92508 [details]
Patch

ok
Comment 4 Dirk Pranke 2011-05-27 19:12:50 PDT 
Committed as r87593.