From: Paul Eggleton <paul.eggleton@linux.intel.com>
To: openembedded-core@lists.openembedded.org
Subject: Re: [RFC PATCH 1/4] qt4: fix compile error in Qt 4.8.0 with gcc 4.6
Date: Fri, 10 Feb 2012 19:31:08 +0000 [thread overview]
Message-ID: <1862774.ijFNRjIgdo@helios> (raw)
In-Reply-To: <1440799.NqLiKZanR0@helios>
On Friday 10 February 2012 13:56:44 Paul Eggleton wrote:
> On Thursday 09 February 2012 13:19:22 Khem Raj wrote:
> > since there is a namespace conflict that should be fixed IMO removing
> > Wrror is not quite the right thing to do here IMO.
>
> So removing -Werror is somewhat heavy-handed, yes; perhaps we can simply
> add -Wno-error=c++0x-compat and mark this patch as inappropriate for
> upstreaming instead?
So, I dug further into this, and I discovered that the reason for this
breaking was that the Qt configure script thought we were not using g++ as the
C++ compiler (!) thus it did not detect the gcc version, and therefore when it
came to the check to see if gcc 4.6+ was being used in order to add the -Wno-
c++0x-compat option to CXXFLAGS (which it already has code to do), the version
was not set and the option was not added.
I've updated the contrib branch to fix this as well as reformat the other patch
as Koen requested.
Cheers,
Paul
--
Paul Eggleton
Intel Open Source Technology Centre
next prev parent reply other threads:[~2012-02-10 19:39 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-09 17:57 [RFC PATCH 0/4] qmake/qt4-tools-nativesdk fixes Paul Eggleton
2012-02-09 17:57 ` [RFC PATCH 1/4] qt4: fix compile error in Qt 4.8.0 with gcc 4.6 Paul Eggleton
2012-02-09 21:19 ` Khem Raj
2012-02-10 13:56 ` Paul Eggleton
2012-02-10 14:03 ` Phil Blundell
2012-02-10 14:09 ` Paul Eggleton
2012-02-10 19:31 ` Paul Eggleton [this message]
2012-02-10 20:36 ` Khem Raj
2012-02-09 17:57 ` [RFC PATCH 2/4] qt4: build qmake for the target Paul Eggleton
2012-02-09 18:18 ` Koen Kooi
2012-02-09 18:22 ` Paul Eggleton
2012-02-09 17:57 ` [RFC PATCH 3/4] qt4-tools-nativesdk: fix build of 4.7.4 Paul Eggleton
2012-02-09 17:57 ` [RFC PATCH 4/4] qt4-tools-nativesdk: improve and tidy up qmake build Paul Eggleton
2012-02-09 18:07 ` [RFC PATCH 0/4] qmake/qt4-tools-nativesdk fixes Otavio Salvador
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1862774.ijFNRjIgdo@helios \
--to=paul.eggleton@linux.intel.com \
--cc=openembedded-core@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox