Bug 112604 - [WinCairo] Build WinCairo WTF and JavaScriptCore Projects under VS2010
Summary: [WinCairo] Build WinCairo WTF and JavaScriptCore Projects under VS2010
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Brent Fulgham
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-03-18 11:41 PDT by Brent Fulgham
Modified: 2013-03-18 16:22 PDT (History)
2 users (show)

See Also:


Attachments
First part of change adding top-level solution labels for Debug/Release_WinCairo (25.19 KB, patch)
2013-03-18 12:01 PDT, Brent Fulgham
thorton: review+
Details | Formatted Diff | Diff
Second part of patch. Land WTF changes. (13.20 KB, patch)
2013-03-18 13:29 PDT, Brent Fulgham
thorton: review+
Details | Formatted Diff | Diff
Third part of patch: Land JavaScriptCore updates. (22.97 KB, patch)
2013-03-18 13:36 PDT, Brent Fulgham
no flags Details | Formatted Diff | Diff
Bad patch. (65.34 KB, patch)
2013-03-18 14:16 PDT, Brent Fulgham
no flags Details | Formatted Diff | Diff
Third part of patch: Land JavaScriptCore changes. (63.48 KB, patch)
2013-03-18 14:23 PDT, Brent Fulgham
thorton: review+
Details | Formatted Diff | Diff
Fix for testapi build under WinCairo (6.13 KB, patch)
2013-03-18 15:54 PDT, Brent Fulgham
thorton: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Brent Fulgham 2013-03-18 11:41:35 PDT
This set of patches updates the VS2010 projects to build the WinCairo target.
Comment 1 Brent Fulgham 2013-03-18 12:01:06 PDT
Created attachment 193623 [details]
First part of change adding top-level solution labels for Debug/Release_WinCairo
Comment 2 Brent Fulgham 2013-03-18 12:02:19 PDT
First patch: Just add a set of do-nothing "WinCairo" build targets.  These will be filled in at specific project-level build settings as needed.  Not all projects will need any special handling, so no project-specific build targets have been created yet.
Comment 3 Brent Fulgham 2013-03-18 13:29:30 PDT
Comment on attachment 193623 [details]
First part of change adding top-level solution labels for Debug/Release_WinCairo

Initial step in landing WinCairo VS2010 build.
Comment 4 Brent Fulgham 2013-03-18 13:29:58 PDT
Created attachment 193639 [details]
Second part of patch. Land WTF changes.
Comment 5 Brent Fulgham 2013-03-18 13:30:56 PDT
Comment on attachment 193639 [details]
Second part of patch. Land WTF changes.

This patch adds Debug_WinCairo/Release_WinCairo targets so headers get copied to proper build folder.
Comment 6 Brent Fulgham 2013-03-18 13:36:24 PDT
Created attachment 193640 [details]
Third part of patch: Land JavaScriptCore updates.
Comment 7 Brent Fulgham 2013-03-18 13:37:17 PDT
Comment on attachment 193640 [details]
Third part of patch: Land JavaScriptCore updates.

This patch gets the WinCairo/CFLite-only build of JavaScriptCore working under VS2010.
Comment 8 Brent Fulgham 2013-03-18 14:11:07 PDT
First patch landed in r146120.
Comment 9 Brent Fulgham 2013-03-18 14:12:08 PDT
Second patch landed in r146121.
Comment 10 Brent Fulgham 2013-03-18 14:16:29 PDT
Created attachment 193648 [details]
Bad patch.
Comment 11 Brent Fulgham 2013-03-18 14:17:44 PDT
Comment on attachment 193648 [details]
Bad patch.

Revised bug to correct jsc and LLInt* projects.
Comment 12 Brent Fulgham 2013-03-18 14:23:49 PDT
Created attachment 193653 [details]
Third part of patch: Land JavaScriptCore changes.
Comment 13 Brent Fulgham 2013-03-18 15:54:05 PDT
Created attachment 193677 [details]
Fix for testapi build under WinCairo
Comment 14 Brent Fulgham 2013-03-18 15:54:45 PDT
Comment on attachment 193677 [details]
Fix for testapi build under WinCairo

Didn't notice that this was still linking against official Apple CoreFoundation.lib. Corrected to use CFLite.lib.
Comment 15 Brent Fulgham 2013-03-18 16:20:53 PDT
Third patch landed in r146123.
Comment 16 Brent Fulgham 2013-03-18 16:21:25 PDT
Testapi build correction landed under r146144.
Comment 17 Brent Fulgham 2013-03-18 16:22:08 PDT
All patches landed.