svn commit: r229800 - head/sys/conf

John Baldwin jhb at freebsd.org
Mon Jan 9 13:48:28 UTC 2012


On Sunday, January 08, 2012 5:43:30 am Pawel Jakub Dawidek wrote:
> On Sun, Jan 08, 2012 at 12:55:22AM +0000, Adrian Chadd wrote:
> > Author: adrian
> > Date: Sun Jan  8 00:55:22 2012
> > New Revision: 229800
> > URL: http://svn.freebsd.org/changeset/base/229800
> > 
> > Log:
> >   Make these two files conditionally build on UFS_ACL, as it doesn't
> >   seem to be used elsewhere.
> >   
> >   Since UFS_ACL is enabled by default for GENERIC kernels, this shouldn't
> >   break anything - but please beat me to fix things if it does.
> 
> If someone is actually using GENERIC kernel. This change will break all
> my system next time I upgrade. Adding UFS_ACL option to the kernel
> config to make ZFS kernel module to work doesn't sound very intuitive.
> 
> I understand what you are trying to accomplish, but we really need to
> find better way to do this. Until then, could you back it out?

Why not make 'options ZFS' work?  Then you can have subr_acl_nfs.c depend on
ZFS as well.

-- 
John Baldwin


More information about the svn-src-all mailing list