From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 23481 invoked by alias); 13 Feb 2008 18:12:48 -0000 Received: (qmail 23458 invoked by uid 22791); 13 Feb 2008 18:12:46 -0000 X-Spam-Check-By: sourceware.org Received: from hq.tensilica.com (HELO mailapp.tensilica.com) (65.205.227.29) by sourceware.org (qpsmtpd/0.31) with ESMTP; Wed, 13 Feb 2008 18:12:29 +0000 Received: from localhost ([127.0.0.1]) by mailapp.tensilica.com with esmtp (Exim 4.34) id 1JPM5m-0007uQ-VG; Wed, 13 Feb 2008 10:12:23 -0800 Received: from mailapp.tensilica.com ([127.0.0.1]) by localhost (mailapp [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 29878-08; Wed, 13 Feb 2008 10:12:22 -0800 (PST) Received: from maxim_fc5.hq.tensilica.com ([192.168.11.68]) by mailapp.tensilica.com with esmtp (Exim 4.34) id 1JPM5m-0007uK-Bo; Wed, 13 Feb 2008 10:12:22 -0800 Message-ID: <47B33305.1090606@hq.tensilica.com> Date: Wed, 13 Feb 2008 18:12:00 -0000 From: Maxim Grigoriev User-Agent: Thunderbird 1.5.0.9 (X11/20070102) MIME-Version: 1.0 To: Markus Deuling CC: Maxim Grigoriev , gdb-patches@sources.redhat.com, Marc Gauthier , Chris Zankel , Bob Wilson Subject: Re: [commit] Adding support for Xtensa GNU/Linux References: <47B0C695.7010908@hq.tensilica.com> <47B298C5.9030001@de.ibm.com> In-Reply-To: <47B298C5.9030001@de.ibm.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2008-02/txt/msg00213.txt.bz2 Markus, Thanks much for fixing this issue. -- Maxim Markus Deuling wrote: > Hi Maxim, > > Maxim Grigoriev schrieb: >> This patch adds support for Xtensa GNU/Linux. >> >> We like XML approach to describe register files. It matches >> Xtensa, which requires register files to be easily configurable. >> >> But, we need more time to update Xtensa FSF GDB/GDBSERVER >> to start fully using this feature. So this version of Xtensa >> FSF GDB/GDBSERVER does not use this new approach. >> > > there is a problem in your patch that breaks GDB's build if you build > with enable-targets=all. > > xtensa-linux-nat.o is part of ALL_TARGET_OBS which are built when > using enable-targets=all. If you now build GDB > for example on x86 you have two native files (the one from x86 and > always xtensa-linux-nat). > > This patch fixes that. Ok to commit? > > ChangeLog: > > * Makefile.in (ALL_TARGET_OBS): Remove dependency to > xtensa-linux-nat.o. > > > > ------------------------------------------------------------------------ > > diff -urpN src/gdb/Makefile.in dev/gdb/Makefile.in > --- src/gdb/Makefile.in 2008-02-12 05:50:17.000000000 +0100 > +++ dev/gdb/Makefile.in 2008-02-13 08:12:06.000000000 +0100 > @@ -479,7 +479,7 @@ ALL_TARGET_OBS = \ > v850-tdep.o \ > vaxnbsd-tdep.o vaxobsd-tdep.o vax-tdep.o \ > xstormy16-tdep.o \ > - xtensa-config.o xtensa-tdep.o xtensa-linux-nat.o xtensa-linux-tdep.o \ > + xtensa-config.o xtensa-tdep.o xtensa-linux-tdep.o \ > glibc-tdep.o \ > bsd-uthread.o \ > nbsd-tdep.o obsd-tdep.o \ >