Bug 11571 - Gdk build is out of sync
Summary: Gdk build is out of sync
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 420+
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-11-11 05:03 PST by Alp Toker
Modified: 2006-11-12 00:57 PST (History)
1 user (show)

See Also:


Attachments
Typo fixes and source file updates to the gdk bakefile (4.67 KB, patch)
2006-11-11 05:06 PST, Alp Toker
mjs: review+
Details | Formatted Diff | Diff
Only include SVGDocumentExtensions.h when SVG_SUPPORT is defined (421 bytes, patch)
2006-11-11 05:09 PST, Alp Toker
mjs: review+
Details | Formatted Diff | Diff
Patch (88.01 KB, patch)
2006-11-12 00:00 PST, Mark Rowe (bdash)
no flags Details | Formatted Diff | Diff
Patch (88.49 KB, patch)
2006-11-12 00:47 PST, Mark Rowe (bdash)
mitz: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.