Bug 22536

Summary: Regression - QtWebKit MinGW port build failure
Product: WebKit Reporter: Laszlo Gombos <laszlo.gombos>
Component: WebKit QtAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: sam, vestbo
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: All   
Attachments:
Description Flags
Fix
vestbo: review-
Revised patch based on comment #4 vestbo: review+

Description Laszlo Gombos 2008-11-27 19:11:55 PST
Build failure in QtWebKit using MinGW port - I have seen the failure on Win using MinGW, but it could inpact other Qt platforms as well. 

Regression introduced in Changeset 36623 and inpacted both the wx and Qt port. Regression in wx port is fixed in Changeset 36717, but QT WebKit regression is still there.
Comment 1 Mark Rowe (bdash) 2008-11-27 19:15:53 PST
It would be more useful if you included some information about what the failure is.
Comment 2 Laszlo Gombos 2008-11-27 19:21:58 PST
Created attachment 25563 [details]
Fix
Comment 3 Laszlo Gombos 2008-11-27 19:25:10 PST
Mark, compiler under MinGW ends up trying to compile .h files, because they are listed as SOURCE files in the WebCore.pro. DO not have the actual error message handy.
Comment 4 Tor Arne Vestbø 2008-11-28 08:41:57 PST
Maybe add the removed headers to HEADERS?
Comment 5 Laszlo Gombos 2008-11-28 09:14:34 PST
Created attachment 25574 [details]
Revised patch based on comment #4

Instead of removing the headers in question from SOURCES, I moved them to HEADERS.
Comment 6 Tor Arne Vestbø 2008-11-28 10:51:48 PST
Good stuff! Landed in r38832