From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([208.118.235.92]:45871)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <blauwirbel@gmail.com>) id 1Sv5qH-0000f4-P2
	for qemu-devel@nongnu.org; Sat, 28 Jul 2012 08:09:58 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <blauwirbel@gmail.com>) id 1Sv5qG-000233-Np
	for qemu-devel@nongnu.org; Sat, 28 Jul 2012 08:09:57 -0400
Received: from mail-yw0-f45.google.com ([209.85.213.45]:45304)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <blauwirbel@gmail.com>) id 1Sv5qG-00022r-JS
	for qemu-devel@nongnu.org; Sat, 28 Jul 2012 08:09:56 -0400
Received: by yhpp34 with SMTP id p34so3808176yhp.4
	for <qemu-devel@nongnu.org>; Sat, 28 Jul 2012 05:09:56 -0700 (PDT)
MIME-Version: 1.0
In-Reply-To: <CAFEAcA-zs=r4+c-ruih0Xf9v6vQcJVv8XNfh0BQDVhqoXw+w3w@mail.gmail.com>
References: <1342620628-12751-1-git-send-email-peter.maydell@linaro.org>
	<1342620628-12751-12-git-send-email-peter.maydell@linaro.org>
	<CAAu8pHsA5A0wmUMMQFJzRvoBU85TFS9nJx6ibWL+MQWDGgCkqw@mail.gmail.com>
	<CAFEAcA-zs=r4+c-ruih0Xf9v6vQcJVv8XNfh0BQDVhqoXw+w3w@mail.gmail.com>
From: Blue Swirl <blauwirbel@gmail.com>
Date: Sat, 28 Jul 2012 12:09:34 +0000
Message-ID: <CAAu8pHu0crC1ZFm=fbYX3gmaavFFZ+sTSCQKTb7j5S+KWJh3zQ@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8
Subject: Re: [Qemu-devel] [PATCH 11/11] configure: Check for -Werror causing
 failures when compiling tests
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: Stefan Weil <sw@weilnetz.de>, qemu-devel@nongnu.org, patches@linaro.org

On Sat, Jul 28, 2012 at 10:57 AM, Peter Maydell
<peter.maydell@linaro.org> wrote:
> On 28 July 2012 10:04, Blue Swirl <blauwirbel@gmail.com> wrote:
>> On Wed, Jul 18, 2012 at 2:10 PM, Peter Maydell <peter.maydell@linaro.org> wrote:
>>> +    echo "ERROR: configure test passed without -Werror but failed with -Werror."
>>> +    echo "This is probably a bug in the configure script. The failing command"
>>> +    echo "will be at the bottom of config.log."
>>> +    echo "You can run configure with --disable-werror to bypass this check."
>>> +    exit 1
>>
>> This should be degraded to a warning
>
> The idea is that by running configure with --enable-werror (either
> explicitly or implicitly for a git build) you've asked for compile
> warnings to cause build failure. We're just doing that in configure
> as well as for the main build.
>
>>, I'm getting configure breakage
>> in some cases:
>
> The thesis of the patch series is that these represent bugs to be fixed.

Yes, but then the errors should be fixed by the series. When adding
new GCC warning flags, I also fixed the causes for the warning first.
Committing known build breakages is not OK.

>
> -- PMM