From owner-freebsd-current Wed Jul 10 17:34:29 2002 Delivered-To: freebsd-current@freebsd.org Received: from mx1.FreeBSD.org (mx1.FreeBSD.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 9A0ED37B400; Wed, 10 Jul 2002 17:34:27 -0700 (PDT) Received: from HAL9000.wox.org (12-233-156-170.client.attbi.com [12.233.156.170]) by mx1.FreeBSD.org (Postfix) with ESMTP id B732643E3B; Wed, 10 Jul 2002 17:34:23 -0700 (PDT) (envelope-from dschultz@uclink.Berkeley.EDU) Received: from HAL9000.wox.org (localhost [127.0.0.1]) by HAL9000.wox.org (8.12.3/8.12.3) with ESMTP id g6B0YVhr001301; Wed, 10 Jul 2002 17:34:31 -0700 (PDT) (envelope-from dschultz@uclink.Berkeley.EDU) Received: (from das@localhost) by HAL9000.wox.org (8.12.3/8.12.3/Submit) id g6B0YUdC001300; Wed, 10 Jul 2002 17:34:30 -0700 (PDT) (envelope-from dschultz@uclink.Berkeley.EDU) Date: Wed, 10 Jul 2002 17:34:24 -0700 From: David Schultz To: Peter Wemm Cc: Matthew Dillon , Dag-Erling Smorgrav , Giorgos Keramidas , current@FreeBSD.ORG Subject: Re: sparc64 tinderbox failure Message-ID: <20020711003424.GA1220@HAL9000.wox.org> Mail-Followup-To: Peter Wemm , Matthew Dillon , Dag-Erling Smorgrav , Giorgos Keramidas , current@FreeBSD.ORG References: <200207102122.g6ALMCi1004528@apollo.backplane.com> <20020710231910.484373808@overcee.wemm.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20020710231910.484373808@overcee.wemm.org> Sender: owner-freebsd-current@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG Thus spake Peter Wemm : > Please do not. gcc is just a tool. If it emits a warning on some arches > because gcc doesn't understand how our libraries work, then we should > disable the gcc checking for those arches on those functions. ie: remove > the __printf0like completely for #ifdef sparc64 for err() etc. ... > This is much much less disruptive than slashing through userland and > "fixing" something that is already perfectly correct and legal. I agree that there's little sense in changing the code to work around a compiler bug, but it does not follow that the change is inherently a bad idea. I tend to think of adding descriptive error messages as something that should have been done long ago, and just happens to be slightly more useful now. The change buys us the time to figure out what's really wrong with gcc, rather than disabling and ignoring the offending warnings on sparc64. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message