Date: Thu, 11 Jul 2002 18:04:42 -0400 From: Mike Barcroft <mike@FreeBSD.org> To: Bruce Evans <bde@zeta.org.au> Cc: Giorgos Keramidas <keramida@FreeBSD.ORG>, current@FreeBSD.ORG Subject: Re: sparc64 tinderbox failure Message-ID: <20020711180442.G89841@espresso.q9media.com> In-Reply-To: <20020712050755.N25069-100000@gamplex.bde.org>; from bde@zeta.org.au on Fri, Jul 12, 2002 at 05:20:12AM %2B1000 References: <20020711005344.F89841@espresso.q9media.com> <20020712050755.N25069-100000@gamplex.bde.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Bruce Evans <bde@zeta.org.au> writes: > On Thu, 11 Jul 2002, Mike Barcroft wrote: > > Comments on the attached, untested patch? > > > Disable fatal warnings during bootstrap, build, and cross tools > > phase of world. > > The setting of NO_WERROR belongs in [BTX]MAKE if anywhere. This is > already done for [BX]MAKE but not for TMAKE. However, I don't like > turning off warnings for any of these. Warnings for these stages may > be even more important and should be less likely than warnings for > building the final world, since it is very important for basic tools > to be correct and for their source to be careful about portabilty > issues. Well, unfortunately I don't think we can depend on older compilers having correct warnings. In PR 40382, it would seem the 4.5 -> HEAD upgrade path is broken because of fatal warnings. A good workaround for that problem might be specifying NO_WERROR for the entire build, in which case -Werror becomes useless anyway. So we might just as well disable early fatal warnings and hope that developers can catch most of the bugs later on in the build. > This also has some style bugs :). Any setting of NO_WERROR turns it on, > so setting it to different spellings of boolean true is just confusing. > It is set correctly for for [BX]MAKE. Oh, that's a much nicer location. :) I think only BMAKE has NO_WERROR defined, not XMAKE. Best regards, Mike Barcroft To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20020711180442.G89841>