Bug 62321 - add a short-flag for --experimental-fully-parallel
Summary: add a short-flag for --experimental-fully-parallel
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: Ojan Vafai
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-08 14:09 PDT by Ojan Vafai
Modified: 2011-06-08 15:54 PDT (History)
5 users (show)

See Also:

Attachments
Patch (1.47 KB, patch)
2011-06-08 14:09 PDT, Ojan Vafai 		no flags 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 Ojan Vafai 2011-06-08 14:09:26 PDT 
add a short-flag for --experimental-fully-parallel
Comment 1 Ojan Vafai 2011-06-08 14:09:42 PDT 
Created attachment 96475 [details]
Patch
Comment 2 Ojan Vafai 2011-06-08 14:10:30 PDT 
This setting is really useful (and reliable) when running just a subset of the tests. It's a pain to have to type all this out everytime.
Comment 3 WebKit Review Bot 2011-06-08 14:38:35 PDT 
Comment on attachment 96475 [details]
Patch

Clearing flags on attachment: 96475

Committed r88388: <http://trac.webkit.org/changeset/88388>
Comment 4 WebKit Review Bot 2011-06-08 14:38:40 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 5 Dirk Pranke 2011-06-08 15:24:58 PDT 
maybe we should get rid of the "--experimental" part at some point?
Comment 6 Ojan Vafai 2011-06-08 15:28:57 PDT 
(In reply to comment #5)
> maybe we should get rid of the "--experimental" part at some point?

I think we should just get rid of the flag entirely if it's stable and doesn't increase flakiness.
Comment 7 Dirk Pranke 2011-06-08 15:54:15 PDT 
(In reply to comment #6)
> (In reply to comment #5)
> > maybe we should get rid of the "--experimental" part at some point?
> 
> I think we should just get rid of the flag entirely if it's stable and doesn't increase flakiness.

I assume "get rid of" == "make it on by default" ? At any rate, it certainly increases flakiness, and I don't know if I would even call it stable (haven't tried it in a while).