[Webkit-unassigned] [Bug 63941] Compilation bug - gcc 4.6.0 and c++0x

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=63941





--- Comment #6 from Ademar Reis <ademar.reis at openbossa.org>  2011-07-21 10:02:19 PST ---
(In reply to comment #5)
> (In reply to comment #3)
> > (From update of attachment 101575 [details] [details])
> > Removing -Wall for everybody is not the right fix, we want -Wall. We have to identify why gcc-4.6.0 is failing and fix the real problem, not hide it. :-)
> > 
> Without the patch, there are two -Wall, the one in QMAKE_CXXFLAG_RELEASE from Fedora is after the -Wno-c++0x-compat already added in QMAKE_CXXFLAGS for GCC4.6.0.
> "-Wall -Wextra ... -Wno-c++0x-compat ... -Wall"
> 

OK, I see what you mean, but still, if -Wall was added somewhere but the intention is to ignore c++0x support warnings (via -Wno-c++0x-compat), then the flags should be introduced together. I mean, whoever is adding -Wall should also add -Wno-c++0x-compat.

Other alternatives are to remove -Werror when building on this configuration or add -Wno-c++0x-compat at the very end.

> So the patch removes second -Wall (the Fedora one), keeping the first (QtWebKit one).

I'm a bit lost here. What do you mean by "the Fedora one"?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.