Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: "Kris Warkentin" <kewarken@qnx.com>
To: "Nick Clifton" <nickc@redhat.com>
Cc: <binutils@sources.redhat.com>, <gdb@sources.redhat.com>
Subject: Re: Adding QNX core-file support to bfd
Date: Fri, 24 Jan 2003 15:08:00 -0000	[thread overview]
Message-ID: <004f01c2c3ba$ae2726d0$2a00a8c0@dash> (raw)
In-Reply-To: <m3u1fz2ali.fsf@north-pole.nickc.cambridge.redhat.com>

> Hi Kris,
>
> > The problem with this is that all of that code and the support functions
for
> > reading our registers, status, etc. is wrapped in #ifdef __QNXTARGET__
which
> > is defined at configure time.  I believe that this is the sort of
clutter
> > that you want to avoid but unfortunately, we rely on some of our
definitions
> > and structures to extract the corefile information.
>
> Using a target specific define does appear to be the way that all core
> file types are supported by the bfd library, so you are not doing
> anything unusual or unwarranted.  Since the header files that allow
> the core file to be decoded are only available on certain targets, it
> makes sense that the inclusion of that particular core file support be
> dependent upon the host/build environment, and this environment is
> determined at configure time.

Okay.  That's good to know.  What I can do to make it a little nicer is to
just put the bare minimum into elf.c and then split our other support
functions out into separate file that will only be compiled for our target.

Second question.  The header file that it requires (nto_debug.h) used to be
in the top level include directory but it seemed to me that any nto specific
includes should be in an 'nto-share' directory in the gdb heirarchy.
Unfortunately, binutils doesn't rely on gdb being present so I'm thinking
that it might need to be in the 'include' dir.  Is there some other way to
do this?  It doesn't look like the include dir is overly cluttered and I
might find some resistance since there aren't really any target specific
headers there.  Perhaps if I just take the definitions I need and put them
into the nto specific file I mentioned earlier?

Kris


  reply	other threads:[~2003-01-24 15:08 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-01-23 22:06 Kris Warkentin
2003-01-24  8:51 ` Nick Clifton
2003-01-24 15:08   ` Kris Warkentin [this message]
2003-01-27 17:19 ` Andrew Cagney
2003-01-27 18:49   ` Kris Warkentin
2003-01-29 17:48     ` Kris Warkentin

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='004f01c2c3ba$ae2726d0$2a00a8c0@dash' \
    --to=kewarken@qnx.com \
    --cc=binutils@sources.redhat.com \
    --cc=gdb@sources.redhat.com \
    --cc=nickc@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