Bug 37251

Summary: Rename target to configuration in new-run-webkit-tests to match the rest of WebKit
Product: WebKit Reporter: Adam Barth <abarth>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: dpranke, eric, ojan
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Attachments:
Description Flags
Patch eric: review+

Description Adam Barth 2010-04-08 00:22:48 PDT
Rename target to configuration in new-run-webkit-tests to match the rest of WebKit
Comment 1 Adam Barth 2010-04-08 00:23:43 PDT
Created attachment 52843 [details]
Patch
Comment 2 Eric Seidel (no email) 2010-04-08 00:27:32 PDT
Comment on attachment 52843 [details]
Patch

Wow, some of this code hurts.

+    optparser.add_option('--release', action='store_const', const='Release',
+                         dest="configuration", help='Set the configuration to Debug')

should say Release.  otherwise r=me.
Comment 3 Adam Barth 2010-04-08 00:28:10 PDT
Committed r57256: <http://trac.webkit.org/changeset/57256>
Comment 4 Eric Seidel (no email) 2010-04-08 00:29:42 PDT
CCing Ojan in case this breaks stuff for the gardner tomrorow.
Comment 5 Adam Barth 2010-04-08 00:50:00 PDT
Added --target back in http://trac.webkit.org/changeset/57258 to fix the downstream bots.  We should change them to use --debug or --release and then remove this evil option.
Comment 6 Dirk Pranke 2010-04-08 12:11:57 PDT
LGTM apart from the thing you already broke and fixed ;)