cvs commit: src/sys/conf options src/sys/i386/acpica Makefile acpi_wakecode.S src/sys/i386/conf NOTES

[Nate Lawson]

On Wed, 29 Oct 2003, M. Warner Losh wrote:
> In message: <20031029100945.F84859 at root.org>
>             Nate Lawson <nate at root.org> writes:
> : On Wed, 29 Oct 2003, M. Warner Losh wrote:
> : > In message: <20031028235425.N82902 at root.org>
> : >             Nate Lawson <nate at root.org> writes:
> : > : Is this needed?  options alone should be sufficient and acpi_wakeup.S
> : > : should include opt_acpi.h.
> : >
> : > Not for kernel modules...
> :
> : Muagh.
> :
> : Ok, I'm starting a pot for whoever will finally implement the combined
> : kernel modules and kernel config build tree.  $10 US via PayPal.  Care to
> : contribute?
>
> Not for this.  Options for kernel compilation should only really be
> used for ABI issues.  better to make this a tunable/sysctl.

Non-sequitur.  The issue still stands that module builds don't pull in the
kernel config file, forcing duplication in make.conf.  When building a
kernel+modules, options in opt_*.h should be included in the module build.
This is even under the 5.2-TODO as "Revised kld build infrastructure".  I
was just indicating that the number of times this has bothered me was
worth $10 to the author of the fix.  Maybe it's bothered enough others
that it's worth a few hundred $ aggregate.  :)

I'll up it to $20.

-Nate