RESOLVED FIXED Bug 124380
[Win] Improve support for command line build of 64-bit JSC 
https://bugs.webkit.org/show_bug.cgi?id=124380
Summary [Win] Improve support for command line build of 64-bit JSC
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.