Bug 11509 – Windows build broken

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Don Gibson]

Patch to be attached shortly.

Depends on bug 11508 to fix some of the breakage.

[Don Gibson]

Created attachment 11370 [details]
patch v1

When applied simultaneously with the patch on bug 11508, this fixes the Windows build.

[Maciej Stachowiak]

I suspect some of this conflicts with http://bugs.webkit.org/show_bug.cgi?id=11433

[Alexey Proskuryakov]

I integrated build fixes from this patch when landing r17592, but could overlook some improvements.

[Don Gibson]

Yeah, this isn't fixed yet; the build is currently broken and I'm pretty sure I fixed at least some of the breakage in this patch.

However, with the landing of bug 11433, I'll need to see what's going on here and whip up a different patch.  Can't work on that until tomorrow, unfortunately.

[Alexey Proskuryakov]

(In reply to comment #4)
> Yeah, this isn't fixed yet; the build is currently broken and I'm pretty sure I
> fixed at least some of the breakage in this patch.

I think the build broke again later - it was OK when I landed bug 11433.

[Don Gibson]

Created attachment 11448 [details]
new patch WIP 1

This doesn't get things building completely, but it goes a long way toward that.  There are some compile errors I haven't figured out yet, plus lots of link-time problems.

This also resurrects the bits from my previous patch that never got committed, some of which aren't really "build bustage" as much as "clean up the vcproj and pull in some of the things that should have been there already".

[Don Gibson]

Created attachment 11450 [details]
new patch WIP 2

Fixes the compile errors by mangling the #include directory order a little less wrongly, cleans up the order of stuff in TemporaryLinkStubs.cpp, adds some missing #include guards, reduces the number of link-time errors

[Don Gibson]

Created attachment 11469 [details]
new patch v1

This gets the Windows port building again, and does some other cleanup as specified in the Changelog.

The build segfaults when run, due mainly to unimplemented FrameLoader() stuff.  Given the size of this patch, I'd prefer to see the build made to work again ASAP; getting it functioning correctly again can follow.

[Don Gibson]

Removing 11508 from the dependency list, as that bug and this bug do not depend on each other any more.

[Don Gibson]

Created attachment 11513 [details]
patch for r17748

Updated the patch to make revision 17748 compile (previous patch was against r17709).

[Don Gibson]

Comment on attachment 11513 [details]
patch for r17748

Oops, forgot to set r?

[Don Gibson]

Created attachment 11522 [details]
patch for r17762

Updated to latest trunk (very few changes needed).

[Don Gibson]

Created attachment 11528 [details]
Patch for r17783

Updated to revision 17783.

[Alexey Proskuryakov]

Tried to land the patch now, but cross-platform code has already changed (ContextMenuController added, many conflicts in TemporaryLinkStubs).

[Don Gibson]

Created attachment 11542 [details]
patch for r17812

Updated to latest trunk.

[Alexey Proskuryakov]

Comment on attachment 11542 [details]
patch for r17812

r=me

[Alexey Proskuryakov]

Committed revision 17816.