>>>>> "Geoff" == Geoff Wing <gcw@pobox.com> writes:

Geoff> This doesn't make sense.  Surely this means you must have
Geoff> already installed apNetBSD.shar (or the pkgsrc
Geoff> sysutils/aperture) otherwise bsd_video.c(*) will bomb out - it
Geoff> won't find "memrange.h" which comes with the aperture package.

Note that on NetBSD 1.5Y and later, since we have kernel support for
MTRR (i.e. HAS_MTRR_BUILTIN is defined), HAS_MTRR_SUPPORT will not be
defined. Thus, "memrange.h" is not included, which means bsd_video.c
builds with no problem on NetBSD-current.

However, breaking the build for 1.5 is not good. We have a few options
for 1.5: either define HasNetBSDApertureDriver to NO for 1.5; or
disable the LKM MTRR support for 1.5, or do something tricky with the
toplevel Makefile to enable the aperture driver support only if the
aperture package is installed.

Which do you think is best here?

- S