svn commit: r200606 - in head/sys: compat/freebsd32 sys

[Doug Barton]

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.


Doug

-- 

	Improve the effectiveness of your Internet presence with
	a domain name makeover!    http://SupersetSolutions.com/