From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 22722 invoked by alias); 3 Nov 2005 22:51:47 -0000 Mailing-List: contact gdb-help@sourceware.org; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-owner@sourceware.org Received: (qmail 22709 invoked by uid 22791); 3 Nov 2005 22:51:43 -0000 Received: from sibelius.xs4all.nl (HELO sibelius.xs4all.nl) (82.92.89.47) by sourceware.org (qpsmtpd/0.30-dev) with ESMTP; Thu, 03 Nov 2005 22:51:43 +0000 Received: from elgar.sibelius.xs4all.nl (root@elgar.sibelius.xs4all.nl [192.168.0.2]) by sibelius.xs4all.nl (8.13.4/8.13.4) with ESMTP id jA3Mp7gI001083; Thu, 3 Nov 2005 23:51:07 +0100 (CET) Received: from elgar.sibelius.xs4all.nl (kettenis@localhost.sibelius.xs4all.nl [127.0.0.1]) by elgar.sibelius.xs4all.nl (8.13.4/8.13.3) with ESMTP id jA3Mp7Zc005058; Thu, 3 Nov 2005 23:51:07 +0100 (CET) Received: (from kettenis@localhost) by elgar.sibelius.xs4all.nl (8.13.4/8.13.4/Submit) id jA3Mp6Ru019811; Thu, 3 Nov 2005 23:51:06 +0100 (CET) Date: Thu, 03 Nov 2005 22:51:00 -0000 Message-Id: <200511032251.jA3Mp6Ru019811@elgar.sibelius.xs4all.nl> From: Mark Kettenis To: drow@false.org CC: brobecker@adacore.com, gdb@sources.redhat.com In-reply-to: <20051103211221.GA11262@nevyn.them.org> (message from Daniel Jacobowitz on Thu, 3 Nov 2005 16:12:21 -0500) Subject: Re: Status on cross builds References: <20051102181756.GB979@adacore.com> <20051102182427.GA2726@nevyn.them.org> <200511022328.jA2NS2ox020148@elgar.sibelius.xs4all.nl> <20051103211221.GA11262@nevyn.them.org> X-SW-Source: 2005-11/txt/msg00081.txt.bz2 > Date: Thu, 3 Nov 2005 16:12:21 -0500 > From: Daniel Jacobowitz > > On Thu, Nov 03, 2005 at 12:28:02AM +0100, Mark Kettenis wrote: > > > Date: Wed, 2 Nov 2005 13:24:27 -0500 > > > From: Daniel Jacobowitz > > > > > > Honestly, I recommend we disable -Werror for this release, if we aren't > > > going to spend a month fixing them. You think the cross builds are > > > bad? It's the natives that really break badly. > > > > But it's only enabled in gdb_mbuild.sh. That hardly matters for the > > release. By default, we still configure without -Werror. I really > > think that's why so many stuff has problems with -Werror. So instead > > I think, we should *enable* -Werror by default on the main branch > > after the release branch has been made. I volunteer to do the > > necessary legwork and create a suitable patch. > > Sorry, we get -Werror by default for the various directories from > binutils. I was close. Ah yes, we might want to switch that off for the release. I think -Werror there caused a few issues on some HP-UX versions, and I'm not sure I fixed all of those. But then there's always --disable-werror. > After a release would be the right time to switch it on for gdb, if > ever. But we would need broad build testing from CVS on the native > platforms, and I don't think we've _ever_ had that. Well, I think it's safe to assume that native platforms that haven't been built during the 6.3 -> 6.4 cycle, will be broken anyway. If we really care about -Werror (and I think we should, bacause it catches sloppy code and real bugs) then the only way to keep our code warning-free is enabling it. That forces people to fix the problems in the code they care about. We can always disable it again shortly before release if we're not confident that all problems have been fixed. Mark