Bug 55050

Summary: [Qt] MinGW build fails to link
Product: WebKit Reporter: Andras Becsi <abecsi>
Component: Tools / TestsAssignee: Andras Becsi <abecsi>
Status: RESOLVED DUPLICATE    
Severity: Blocker CC: jturcotte, laszlo.gombos, ossy
Priority: P1 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Windows XP   
Bug Depends on:    
Bug Blocks: 50251    
Attachments:
Description Flags
proposed patch
none
proposed patch
none
build log for linking none

Andras Becsi
Reported 2011-02-23 08:35:32 PST
After the recent build system changes the MinGW linking fails with undefined reference errors. Apparently the compiler does not like the current library order.
Attachments
proposed patch (11.85 KB, patch)
2011-02-23 08:57 PST, Andras Becsi 		no flags
DetailsFormatted DiffDiff
proposed patch (10.60 KB, patch)
2011-02-23 09:20 PST, Andras Becsi 		no flags
DetailsFormatted DiffDiff
build log for linking (5.62 KB, text/plain)
2011-03-23 09:09 PDT, Csaba Osztrogonác 		no flags
Details
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Andras Becsi
Comment 1 2011-02-23 08:41:39 PST
This documentation says: http://www.network-theory.co.uk/docs/gccintro/gccintro_18.html "Most current linkers will search all libraries, regardless of order, but since some do not do this it is best to follow the convention of ordering libraries from left to right." The problem win-gcc has is that the webcore and jscore libraries are after the windows libraries in the build command line, so there are unresolved symbols.
Andras Becsi
Comment 2 2011-02-23 08:57:08 PST
Created attachment 83488 [details] proposed patch I don't set r?, so Laszlo can check whether it works on Symbian, Ossy already shut down our Windows bots.
Andras Becsi
Comment 3 2011-02-23 09:20:46 PST
Created attachment 83490 [details] proposed patch Fix the final linking order to be webkit2 webcore jscore.
Laszlo Gombos
Comment 4 2011-02-23 20:56:32 PST
(In reply to comment #3) > Created an attachment (id=83490) [details] > proposed patch > > Fix the final linking order to be webkit2 webcore jscore. No regression on Symbian build from this patch. Patch looks good to me, please put it up for review.
Andras Becsi
Comment 5 2011-02-24 02:06:46 PST
Comment on attachment 83490 [details] proposed patch Thanks, Laszlo, for checking.
Laszlo Gombos
Comment 6 2011-02-24 04:49:35 PST
Comment on attachment 83490 [details] proposed patch Looks good to me, r+.
Andras Becsi
Comment 7 2011-02-24 05:57:05 PST
Landed in http://trac.webkit.org/changeset/79548. Closing bug.
Andras Becsi
Comment 8 2011-02-24 05:57:33 PST
Comment on attachment 83490 [details] proposed patch Clearing flags.
Csaba Osztrogonác
Comment 9 2011-03-23 09:07:41 PDT
Reopen, because it still fails with MinGW on Windows. (But works with cross-MinGW on Linux - with same MinGW version)
Csaba Osztrogonác
Comment 10 2011-03-23 09:09:12 PDT
Created attachment 86626 [details] build log for linking
Laszlo Gombos
Comment 11 2011-03-23 19:22:26 PDT
I prefer to close this as DUP as the other bug was opened earlier for MingW on Windows (even though that bug has a higher bug id). There is also some good progress on resolving this on the other bug. *** This bug has been marked as a duplicate of bug 56723 ***
Note You need to log in before you can comment on or make changes to this bug.