Date: 25 Jun 2000 01:57:20 +0200 From: Assar Westerlund <assar@sics.se> To: Warner Losh <imp@village.org> Cc: Garrett Wollman <wollman@khavrinen.lcs.mit.edu>, current@FreeBSD.ORG, kris@FreeBSD.ORG Subject: Re: vnode_if.h: how should it be done ? Message-ID: <5lwvjetyxr.fsf@assaris.sics.se> In-Reply-To: Warner Losh's message of "Sat, 24 Jun 2000 00:10:14 -0600" References: <5llmzyqsfm.fsf@assaris.sics.se> <5ln1kesdkq.fsf@assaris.sics.se> <200006220345.XAA42447@khavrinen.lcs.mit.edu> <200006240610.AAA01831@harmony.village.org>
index | next in thread | previous in thread | raw e-mail
Warner Losh <imp@village.org> writes:
> In message <5llmzyqsfm.fsf@assaris.sics.se> Assar Westerlund writes:
> : The problem is that the source files are hidden in the kernel source
> : directory and not installed. Where should vnode_if.{src,pl} get
> : installed? It seems much simpler just to install vnode_if.h in
> : /usr/include/sys. Patch appended.
>
> No.
> We don't install kenrel internal files.
But it's not kernel internal. It's used by modules/KLDs ? Where
should these expect to find it?
> Use SYSDIR instead.
I'm not sure what you mean by this? Should vnode_if.h just reside in
/sys/sys and be used from there or what?
> : + ${INSTALL} -C -o ${BINOWN} -g ${BINGRP} -m 444 \
> : + ${.OBJDIR}/vnode_if.h \
> : + ${DESTDIR}/usr/include/sys
>
> I object to this part of the patch. We shouldn't install this file.
> It isn't needed by anything outside of the kernel/drivers.
Well, depends on your point of view if that's outside kernel/drivers,
but it's sure used by modules/KLDs.
> For drivers, it should be built automatically as part of the modules
> build system ala the *_if.m stuff.
Where are these files installed so that they can be used by
third-party KLDs?
/assar
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-current" in the body of the message
help
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?5lwvjetyxr.fsf>