Date: 30 Jan 2003 16:10:21 +1100 From: Benno Rice <benno@FreeBSD.org> To: current@FreeBSD.org Subject: Re: Patch to teach config(8) about "platforms". Message-ID: <1043903342.613.17.camel@localhost> In-Reply-To: <20030130045740.GC5754@dragon.nuxi.com> References: <20030128182013.A13422@FreeBSD.org> <20030129025124.GG1016@athlon.pn.xcllnt.net> <20030128190158.A15778@FreeBSD.org> <20030129044548.GI1016@athlon.pn.xcllnt.net> <20030128205737.A22274@FreeBSD.org> <20030129051853.GJ1016@athlon.pn.xcllnt.net> <1043819769.648.52.camel@localhost> <20030129062558.GB1715@athlon.pn.xcllnt.net> <1043821970.648.60.camel@localhost> <20030129074647.GD1715@athlon.pn.xcllnt.net> <20030130045740.GC5754@dragon.nuxi.com>
next in thread | previous in thread | raw e-mail | index | archive | help
[-- Attachment #1 --]
On Thu, 2003-01-30 at 15:57, David O'Brien wrote:
> On Tue, Jan 28, 2003 at 11:46:47PM -0800, Marcel Moolenaar wrote:
> > I would not introduce a <platform/foo.h>, but rather
> > <machine/${variant}/foo.h>. The reason for this is that the
> > /usr/include/platform directory is only needed on powerpc and mips,
> > which seems to indicate that it should be under <machine>. Also,
> > the use of machine/${variant} allows us to install the headers for
> > all variants, which may improve cross-building.
>
> This is a very nice way of adding the complexity when it is needed, but
> not getting in the way of those cases where it isn't needed.
Except that it requires us to know which platform variant directory to
use in each case. If this directory is then copied or symlinked to
/usr/include/platform or /usr/include/machine/platform, I don't object
but otherwise it'll lead to an ifdef nightmare in the machine includes.
--
Benno Rice <benno@FreeBSD.org>
[-- Attachment #2 --]
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.1 (FreeBSD)
iD8DBQA+OLNuXjRwWofFmQkRAuIfAKCDMWTadE6yWCDpbjJRBvChfnA4mwCfSK6m
7XWNNZP+uTjjeIz+0KZFTUk=
=u1M1
-----END PGP SIGNATURE-----
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1043903342.613.17.camel>