This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [PATCH] GNU/k*BSD fixes [w/ChangeLog]
On Mon, Aug 09, 2004 at 07:48:33PM +0200, Mark Kettenis wrote:
>
> (note: osreldate.h and sys/param.h are equivalent for this purpose)
>
> Indeed <sys/param.h> defenitely exists on all FreeBSD systems, and is
> the official way to get the defined we're talking about here.
Ok. I'll take note on this.
> Not really. I'm sort of bending the rules in i386bsd-nat.c by
> checking the version numbers. By adding more to it, I run the risk
> that people start noticing it ;-). Anyway, this code is not really
> adding any functionality. It's just a bunch of consistency checks,
> that you can perfectly well do without.
>
> [...]
>
> I'm talking about dropping support for pre-FreeBSD-BSD here, which
> actually isn't supported. I'll remove that stuff and you can forget
> about the #ifdefs.
But the macro checks indicated the check is for FreeBSD 3.x? Well, I'm
probably missing something here..
> We have libkvm, but not nlist() which is not part of Glibc. I'm considering
> adding nlist() and other functions in a compatibility library (e.g. libbsd),
> though.
>
> IIRC the Hurd once had such a compatibility library. Anyway ...
It was part of Glibc. Then it was renamed to libutil and still exists, but
with a slightly different purpose.
> But I was surprised that disabling the '#include <nlist.h>' didn't cause the
> build process to fail on missing declarations. Does bsd-kvm.c really work
> without nlist, or am I missing something?
>
> It only needs the definition of `struct nlist'. If something else
> provides that (presumably your <kvm.h> has been hacked to do that)
Heh. Actualy I hacked kvm.h myself to do precisely this (through inclussion
of some kernel headers). I just didn't remember. =)
> everything is fine. I'll add the #ifdef HAVE_NLIST_H. It can't hurt.
Ok. Please don't forget about the hunks for gdb/configure.{host,tgt}.
--
Robert Millan
(Debra and Ian) (Gnu's Not (UNiplexed Information and Computing System))/\
(kernel of *(Berkeley Software Distribution))