Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Nick Hudson <skrll@netbsd.org>
To: Daniel Jacobowitz <drow@false.org>
Cc: gdb-patches@sources.redhat.com;
Subject: Re: PATCH: Provde a pid to filename conversion for NetBSD
Date: Sat, 16 Dec 2006 08:46:00 -0000	[thread overview]
Message-ID: <200612160846.31929.skrll@netbsd.org> (raw)
In-Reply-To: <20061215163730.GA9822@nevyn.them.org>

On Friday 15 December 2006 16:37, Daniel Jacobowitz wrote:
> On Fri, Dec 15, 2006 at 04:23:30PM +0000, Nick Hudson wrote:
> > > 2006-11-30  Nick Hudson  <skrll@netbsd.org>
> > >
> > >         * i386nbsd-nat.c: Include "nbsd-nat.h".
> > >         (_initialize_i386nbsd_nat): Update target vector to use
> > >         nbsd_pid_to_exec_file.
> > >
> > >         * config/i386/nbsdelf.mh (NATDEPFILES): Add nbsd-nat.o
> > >
> > >         * nbsd-nat.c: New file.
> > >         * nbsd-nat.h: New file.
> > >         * Makefile.in (ALLDEPFILES): Add nbsd-nat.c.
> > >         (nbsd_nat_h): New variable.
> > >         (nbsd-nat.o): New dependency.
> >
> > What needs to happen for this to go in?
>
> Someone needs to review it.  I was hoping that someone who actually
> used NetBSD would speak up :-)  If no one does, I'll do it myself.

I sent Jason Thorpe (current NetBSD maintainer) an email, but got not 
response. :(

> It looks generally fine.  Is this going to work for every NetBSD
> platform?  Should it be added to all of them, rather than just i386?
> Looks like GDB supports 11.

I have diffs to add support to all other NetBSD platforms which I committed to 
the NetBSD tree recently. I'll send those after this one goes in.

Thanks,
Nick


  reply	other threads:[~2006-12-16  8:46 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-11-30 13:23 Nick Hudson
2006-12-15 16:23 ` Nick Hudson
2006-12-15 16:37   ` Daniel Jacobowitz
2006-12-16  8:46     ` Nick Hudson [this message]
2006-12-16 11:18       ` Mark Kettenis
2006-12-16 17:03         ` Daniel Jacobowitz
2006-12-17 14:32           ` Mark Kettenis

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200612160846.31929.skrll@netbsd.org \
    --to=skrll@netbsd.org \
    --cc=drow@false.org \
    --cc=gdb-patches@sources.redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox