Bug 61957

Summary: [Qt] Build fix of QtWebKit 2.2 when inside Qt tree with GCC 4.6.
Product: WebKit Reporter: Alexis Menard (darktears) <menard>
Component: WebKit QtAssignee: Alexis Menard (darktears) <menard>
Status: RESOLVED FIXED    
Severity: Blocker CC: ademar, dihan.wickremasuriya, ossy
Priority: P1 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
Patch
none
Patch benjamin: review+, benjamin: commit-queue+

Description Alexis Menard (darktears) 2011-06-02 13:27:14 PDT
[Qt] Build fix of QtWebKit 2.2 when inside Qt tree with GCC 4.6.
Comment 1 Alexis Menard (darktears) 2011-06-02 13:35:11 PDT
Created attachment 95799 [details]
Patch
Comment 2 Andreas Kling 2011-06-02 13:37:17 PDT
Comment on attachment 95799 [details]
Patch

r=me, brah
Comment 3 Alexis Menard (darktears) 2011-06-02 13:38:22 PDT
Committed r87950: <http://trac.webkit.org/changeset/87950>
Comment 4 Csaba Osztrogonác 2011-06-02 15:46:17 PDT
(In reply to comment #3)
> Committed r87950: <http://trac.webkit.org/changeset/87950>

It broke Mac build. :-/
cc1plus: error: unrecognized command line option "-Wno-c++0x-compat"

Reopen to fix it.
Comment 5 Alexis Menard (darktears) 2011-06-03 02:20:12 PDT
(In reply to comment #4)
> (In reply to comment #3)
> > Committed r87950: <http://trac.webkit.org/changeset/87950>
> 
> It broke Mac build. :-/
> cc1plus: error: unrecognized command line option "-Wno-c++0x-compat"
> 
> Reopen to fix it.

How is it possible? Mac runs GCC 4.6.0?
Comment 6 Csaba Osztrogonác 2011-06-03 02:35:19 PDT
I don't know ... Dihan?
Comment 7 Alexis Menard (darktears) 2011-06-03 03:55:23 PDT
And it's protected in a linux { } condition so I close it unless someone show me exactly what's going on.
Comment 8 Alexis Menard (darktears) 2011-06-03 04:48:16 PDT
Created attachment 95888 [details]
Patch
Comment 9 Benjamin Poulain 2011-06-03 04:57:50 PDT
Comment on attachment 95888 [details]
Patch

Looks ok. Let's see if the bot likes it :)
Comment 10 Alexis Menard (darktears) 2011-06-03 05:04:53 PDT
Committed r88006: <http://trac.webkit.org/changeset/88006>
Comment 11 Ademar Reis 2011-06-03 06:44:38 PDT
Revision r87950 cherry-picked into qtwebkit-2.2 with commit cf23541 <http://gitorious.org/webkit/qtwebkit/commit/cf23541>
Revision r88006 cherry-picked into qtwebkit-2.2 with commit 5902ab9 <http://gitorious.org/webkit/qtwebkit/commit/5902ab9>
Comment 12 Dihan Wickremasuriya 2011-06-03 08:49:29 PDT
(In reply to comment #6)
> I don't know ... Dihan?

I know this issue has been resolved, but in case anyone still needs this info the version of gcc/g++ on the Mac build slave is 4.2.1 build 5664.

Detailed software/hardware specs for the Mac slave are now available on the bot:
http://build.webkit.sed.hu/buildslaves/szeged-xserve-1