Bug 11571

Summary: Gdk build is out of sync
Product: WebKit Reporter: Alp Toker <alp>
Component: WebCore Misc.Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: mrowe
Priority: P2    
Version: 420+   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
Typo fixes and source file updates to the gdk bakefile
mjs: review+
Only include SVGDocumentExtensions.h when SVG_SUPPORT is defined
mjs: review+
Patch
none
Patch mitz: review+

Description Alp Toker 2006-11-11 05:03:04 PST
The gdk port of WebKit appears to have gone out of sync with the rest of the platform. Attached are an update the existing bakefile rules and other related fixes.

Note that these fixes do not yet make it possible to run the launcher test app, but they provide a basis that can continue to be worked upon incrementally.

This should not be considered an alternative to merging the cmake build system, but rather a stop-gap solution that will allow work on the gdk port to continue in the meantime.
Comment 1 Alp Toker 2006-11-11 05:06:11 PST
Created attachment 11483 [details]
Typo fixes and source file updates to the gdk bakefile
Comment 2 Alp Toker 2006-11-11 05:09:49 PST
Created attachment 11484 [details]
Only include SVGDocumentExtensions.h when SVG_SUPPORT is defined
Comment 3 Maciej Stachowiak 2006-11-11 09:35:58 PST
Comment on attachment 11484 [details]
Only include SVGDocumentExtensions.h when SVG_SUPPORT is defined

r=me
Comment 4 Maciej Stachowiak 2006-11-11 09:36:01 PST
Comment on attachment 11483 [details]
Typo fixes and source file updates to the gdk bakefile

r=me
Comment 5 Mark Rowe (bdash) 2006-11-12 00:00:42 PST
Created attachment 11489 [details]
Patch

This patch includes the contents of attachment 11483 [details] and attachment 11484 [details], but goes further to allow JSCore and WebCore to compile and link sucessfully.  GdkLauncher does not link against WebCore due to two dozen undefined symbols that are related to the loader and screen refactorings from this week.
Comment 6 Mark Rowe (bdash) 2006-11-12 00:47:50 PST
Created attachment 11491 [details]
Patch

Remove some commented out lines of code.
Comment 7 mitz 2006-11-12 00:48:59 PST
Comment on attachment 11491 [details]
Patch

r=me
Comment 8 Mark Rowe (bdash) 2006-11-12 00:57:29 PST
Landed in r17741.  WebCore should now compile + link correctly, though it will fail with unresolved symbols when being linked into GdkLauncher.  Patches for that should be attached to a new bug report.