re: CVS commit: [pgoyette-compat] src/sys/arch/amd64/conf

[matthew green <mrg%eterna.com.au@localhost>]

"Paul Goyette" writes:
> Module Name:	src
> Committed By:	pgoyette
> Date:		Tue Mar  6 04:45:04 UTC 2018
> 
> Modified Files:
> 	src/sys/arch/amd64/conf [pgoyette-compat]: GENERIC
> Added Files:
> 	src/sys/arch/amd64/conf [pgoyette-compat]: NOCOMPAT
> 
> Log Message:
> Add a NOCOMPAT kernel definition to serve as a baseline for getting
> the incremental COMPAT_xx options working.
> 
> While here, add some additional "requires COMPAT_xxx" to some of the
> options in GENERIC

thanks for fixing this stuff up.  i think there can be more
clean up here.  eg:

! options       COMPAT_LINUX32  # requires COMPAT_LINUX and COMPAT_NETBSD32
!                               # and COMPAT_50 (or older)

this sort of thing shoudd be handled by having depends in
the files* files.  eg,

conf/files:defflag    opt_compat_netbsd.h     COMPAT_70: COMPAT_NETBSD

should be all it needs, not any comments or extra options.

thanks.


.mrg.