From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 5734 invoked by alias); 2 Nov 2005 14:54:26 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 5723 invoked by uid 22791); 2 Nov 2005 14:54:23 -0000 Received: from nevyn.them.org (HELO nevyn.them.org) (66.93.172.17) by sourceware.org (qpsmtpd/0.30-dev) with ESMTP; Wed, 02 Nov 2005 14:54:23 +0000 Received: from drow by nevyn.them.org with local (Exim 4.54) id 1EXK0K-0007Q9-7T; Wed, 02 Nov 2005 09:54:20 -0500 Date: Wed, 02 Nov 2005 18:40:00 -0000 From: Daniel Jacobowitz To: Andrew STUBBS Cc: gdb-patches@sources.redhat.com Subject: Re: PATCH: Build sh-elf with -Werror Message-ID: <20051102145420.GA28462@nevyn.them.org> Mail-Followup-To: Andrew STUBBS , gdb-patches@sources.redhat.com References: <4368D237.8040704@st.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4368D237.8040704@st.com> User-Agent: Mutt/1.5.8i X-SW-Source: 2005-11/txt/msg00038.txt.bz2 On Wed, Nov 02, 2005 at 02:50:31PM +0000, Andrew STUBBS wrote: > Hi all, > > Following my recent mistake with the -batch-silent patch I thought I had > better check I hadn't missed anything else due to my forgetting > '--enable-gdb-build-warnings=,-Werror'. > > I found that I had not, but that the sh-elf debugger wouldn't build due > to pre-existing problems. I guess nobody is interested in sh-elf or > everybody forgets Werror. The gdb_byte warnings mostly only show up with gcc4. > The attached patch fixes the problems. Note that monitor.c actually did > build because it is never built with Werror, but not all the warnings > were unavoidable. These are OK. Thanks. -- Daniel Jacobowitz CodeSourcery, LLC