From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 26656 invoked by alias); 30 Jun 2002 19:05:08 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 26642 invoked from network); 30 Jun 2002 19:05:04 -0000 Received: from unknown (HELO crack.them.org) (65.125.64.184) by sources.redhat.com with SMTP; 30 Jun 2002 19:05:04 -0000 Received: from gruel-2-149.ppp.andrew.cmu.edu ([128.2.2.149] helo=nevyn.them.org) by crack.them.org with asmtp (Exim 3.12 #1 (Debian)) id 17Ok09-0003D4-00; Sun, 30 Jun 2002 14:04:49 -0500 Received: from drow by nevyn.them.org with local (Exim 3.35 #1 (Debian)) id 17Ojzw-00016n-00; Sun, 30 Jun 2002 15:04:36 -0400 Date: Sun, 30 Jun 2002 12:05:00 -0000 From: Daniel Jacobowitz To: David O'Brien Cc: Mark Kettenis , gdb-patches@sources.redhat.com Subject: Re: Include param.h on FreeBSD/i386 Message-ID: <20020630190436.GA4210@nevyn.them.org> Mail-Followup-To: David O'Brien , Mark Kettenis , gdb-patches@sources.redhat.com References: <20020628131544.A53773@dragon.nuxi.com> <86k7og24w7.fsf@elgar.kettenis.dyndns.org> <20020630120003.B70260@dragon.nuxi.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20020630120003.B70260@dragon.nuxi.com> User-Agent: Mutt/1.5.1i X-SW-Source: 2002-06/txt/msg00588.txt.bz2 On Sun, Jun 30, 2002 at 12:00:03PM -0700, David O'Brien wrote: > On Sun, Jun 30, 2002 at 04:54:48PM +0200, Mark Kettenis wrote: > > "David O'Brien" writes: > > > > > I applied this patch to both mainline and 5.2_branch: > > > > > > 2002-06-28 David O'Brien > > > > > > * config/i386/nm-fbsd.h: Include . > > > * config/i386/tm-fbsd.h: Likewise. > > > > Hi David. Can you tell me why this is needed? > > Some code I added to GDB 5.2 in the FreeBSD source tree needs it. That > code isn't ready to submit back yet. Since the include is benign but I > wanted to reduce the diffs where easy. I actually don't need it in > nm-fbsd.h any more. > > > Including > > from config/i386/tm-fbsd.h isn't right since this file > > is going to be removed in the not so distant future, > > No problem, I'll reimpliment when that happens. Including system headers from the tm file is always pretty dodgy - what do you need it for? How does that include affect cross-debuggers? -- Daniel Jacobowitz Carnegie Mellon University MontaVista Software Debian GNU/Linux Developer