Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 18 Mar 1995 00:44:05 -0800 (PST)
From:      "Rodney W. Grimes" <rgrimes@gndrsh.aac.dev.com>
To:        bde@zeta.org.au (Bruce Evans)
Cc:        davidg@Root.COM, phk@ref.tfs.com, CVS-commiters@freefall.cdrom.com, cvs-sys@freefall.cdrom.com
Subject:   Re: cvs commit: src/sys/libkern locc.c random.c scanc.c skpc.c libkern
Message-ID:  <199503180844.AAA20825@gndrsh.aac.dev.com>
In-Reply-To: <199503180835.SAA13231@godzilla.zeta.org.au> from "Bruce Evans" at Mar 18, 95 06:35:29 pm

next in thread | previous in thread | raw e-mail | index | archive | help
> 
> >>>   We install <sys/systm.h>, it referenced <libkern/libkern.h> which
> >>>   wasn't there (unless you have sources installed).  this is wrong.
> >>> 
> >>> An alternative solution might be: "#ifdef KERNEL" around the include.
> 
> No, <sys/systm.h> shouldn't be included by non-kernel code at all.

And my grepping of the source tree turned up no one that directly
inclues sys/systm.h, but it *is* included by vm_page.h.  I have
yet to search the source tree for that one.

This does bring up the question of should we be installing all the
files we are in /usr/include/sys.  As I am about to rewrite the
/usr/include Makefile to clean up all the mkdir/chown/chmod this
would be a *very* good time to clean up *exactly* what header
files we install into /usr/include/sys.

> >>Not as long as we install /usr/lib/libkern.a then we simply need to
> >>install libkern.h as well.
> 
> >   We shouldn't install libkern.a.
> 
> I agree.

If no one objects by tomarrow afternoon we will not be installing
libkern* in /usr/lib.  This is just a waste of (albiet it small)
disk space.


-- 
Rod Grimes                                      rgrimes@gndrsh.aac.dev.com
Accurate Automation Company                   Custom computers for FreeBSD



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199503180844.AAA20825>