Re: [PATCH] GNU/k*BSD fixes [w/ChangeLog]

[Robert Millan <rmh@debian.org>]

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))