Bug 124380

Summary: [Win] Improve support for command line build of 64-bit JSC
Product: WebKit Reporter: Brent Fulgham <bfulgham>
Component: Tools / TestsAssignee: Brent Fulgham <bfulgham>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, commit-queue, dbates, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch ddkilzer: review+

Brent Fulgham
Reported 2013-11-14 14:27:39 PST
The existing build scripts improperly include the "|x64" suffix to the build configuration when chaining scripts together, such as when using "run-javascriptcore-tests". This patch removes this problem so that 32-bit and 64-bit JSC build and tests can be run from the command line.
Attachments
Patch (2.60 KB, patch)
2013-11-14 14:31 PST, Brent Fulgham 		ddkilzer: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Brent Fulgham
Comment 1 2013-11-14 14:31:21 PST
Created attachment 216981 [details] Patch
Radar WebKit Bug Importer
Comment 2 2013-11-14 14:34:49 PST
<rdar://problem/15473874>
David Kilzer (:ddkilzer)
Comment 3 2013-11-14 16:26:24 PST
Comment on attachment 216981 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=216981&action=review r=me! > Tools/Scripts/webkitdirs.pm:377 > + push(@args, '--debug') if ($configuration =~ "Debug"); > + push(@args, '--release') if ($configuration =~ "Release"); Nit: Maybe match on "^Debug" and "^Release" Or do you want other (less valid) things to work as well?
Brent Fulgham
Comment 4 2013-11-14 20:56:05 PST
Committed r159328: <http://trac.webkit.org/changeset/159328>
Brent Fulgham
Comment 5 2013-11-14 20:56:49 PST
I incorporated David's improvement.
Note You need to log in before you can comment on or make changes to this bug.