svn commit: r200606 - in head/sys: compat/freebsd32 sys
M. Warner Losh
imp at bsdimp.com
Tue Dec 22 03:50:50 UTC 2009
In message: <4B2C127A.8060509 at FreeBSD.org>
Doug Barton <dougb at FreeBSD.org> writes:
: Dag-Erling Smørgrav wrote:
: > John Baldwin <jhb at freebsd.org> writes:
: >> Warner Losh <imp at freebsd.org> writes:
: >>> Log:
: >>> Fix compiling FREEBSD_COMPAT[4,5,6] without FREEBSD_COMPAT7.
: >>>
: >>> Note: Not sure this is the right way to do compat, but it makes the
: >>> headers consistent with the implementations.
: >> Please revert. I think this has already been reverted once before.
: >
: > Yep, that was me... we concluded back then that this is a perfectly
: > reasonable dependency; perhaps we should simply add the appropriate
: > #ifndef / #error constructs so you can't define FREEBSD_COMPAT(n)
: > without FREEBSD_COMPAT(n+1).
:
: Trying to look at this from the user perspective, it's kind of silly
: that if what I want is COMPAT5 I also need to add COMPAT<somethingelse>.
:
: I don't know the issues in the code so I don't know what the solution
: should be on the back end, but on the front end what needs to work is
: that I specify the compat level I want in my kernel config, then the
: code does what's necessary to make that work.
The trouble is that we need to do this globally, or certain
definitions don't get defined. It is easier to just #error out than
hack config to do the right thing I think...
Warner
More information about the svn-src-all
mailing list