Bug 25950 - JavaScriptCore Fails to Build on Windows (Cairo)
Summary: JavaScriptCore Fails to Build on Windows (Cairo)
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore JavaScript (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-05-21 16:21 PDT by Brent Fulgham
Modified: 2009-05-22 16:01 PDT (History)
1 user (show)

See Also:


Attachments
Patch to allow redistributable JavaScriptCore Builds (9.70 KB, patch)
2009-05-22 12:57 PDT, Brent Fulgham
no flags Details | Formatted Diff | Diff
Revised patch to use CFLite labels. (12.17 KB, patch)
2009-05-22 13:23 PDT, Brent Fulgham
no flags Details | Formatted Diff | Diff
Correct my name, remove erroneous Debug_Cairo/Release_Cairo references. (12.16 KB, patch)
2009-05-22 13:38 PDT, Brent Fulgham
sfalken: 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 2009-05-21 16:21:23 PDT
The recent (great!) change to using more Visual Studio property sheets introduced a build error for the non-Apple build of JavaScriptCore.  The Cairo version links against CFLite.dll, rather than CoreFoundation (as this is not redistributable).

Unfortunately, the JavaScriptCoreCommon.vsprops file assumes full CoreFoundation support.

Suggestion:
Create two new property sheets, and modify the projects to make use of them:
(1) JavaScriptCoreCF.vsprops, which uses the true CoreFoundation library.
(2) JavaScriptCoreCFLite.vsprops, which uses the open source CFLite library.

All current targets in the JavaScriptCore project will now also inherit from JavaScriptCoreCF.vsprops, and should be unaffected.

Create two new project targets (Debug_Cairo and Release_Cairo), that will be identical to the standard Debug and Release, but will inherit from JavaScriptCoreCFLite.vsprops instead of JavaScriptCoreCF.vsprops.
Comment 1 Brent Fulgham 2009-05-22 12:57:32 PDT
Created attachment 30589 [details]
Patch to allow redistributable JavaScriptCore Builds
Comment 2 Steve Falkenburg 2009-05-22 13:05:00 PDT
Please consider using _CFLite as the suffix for the configs instead of Cairo, since JavaScriptCore has no Cairo dependencies.
Comment 3 Brent Fulgham 2009-05-22 13:23:54 PDT
Created attachment 30592 [details]
Revised patch to use CFLite labels.
Comment 4 Steve Falkenburg 2009-05-22 13:29:43 PDT
You need to use your name instead of "U-bfulgham-PC\bfulgham" in both ChangeLogs.
Config is still named Debug_Cairo/Release_Cairo. Please rename to Debug_CFLite/Release_CFLite.
Fix ChangeLog to refer to Debug_CFLite/Release_CFLite instead of _Cairo.
Comment 5 Brent Fulgham 2009-05-22 13:38:01 PDT
Created attachment 30593 [details]
Correct my name, remove erroneous Debug_Cairo/Release_Cairo references.

Minor update to replace U-bfulgham-PC\bfulgham with Brent Fulgham, as well as correctly naming configurations.
Comment 6 Brent Fulgham 2009-05-22 16:01:00 PDT
Landed in r44063.