2004-12-04 22:26:53

by Chip Salzenberg

[permalink] [raw]
Subject: ACL patch - is this bad or good?

Hi there. A Debian user contributed a copy of what he says is a Red
Hat patch to support "acl" and "no_acl" options for NFS exports.

Is this patch OK?
--
Chip Salzenberg - a.k.a. - <[email protected]>
People are supposed to die for freedom. Not the other way around.


Attachments:
(No filename) (291.00 B)
nfs-utils-1.0.3-aclexport.patch (3.31 kB)
Download all attachments

2004-12-06 00:31:43

by NeilBrown

[permalink] [raw]
Subject: Re: ACL patch - is this bad or good?

On Saturday December 4, [email protected] wrote:
> Hi there. A Debian user contributed a copy of what he says is a Red
> Hat patch to support "acl" and "no_acl" options for NFS exports.
>
> Is this patch OK?

Define "OK" ???

I must admit a personal bias. I find it very hard to get enthusiastic
about anything relating to ACLs. I think they are one of the worst
mis-features ever to be grafted on to the side of a filesystem.

I am *very* tempted to say "Don't use an old client if your server
supports ACLs", but there are probably some poor souls out there who
are forced to do so due to circumstances beyond their control......

This patch is OK as far as it goes. However it is completely useless
against a kernel.org kernel as support for NFSEXP_NOACL hasn't been
merged yet.

But I suspect it will be if anyone bothers to pester me about it .....

So I am OK with this patch going in to nfs-utils, providing the man
page make it clear that not all kernels support this functionality,
and will quietly accept but ignore the flag.

Anyone want to re-make the patch with a suitable change to the
manpage?

NeilBrown


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://productguide.itmanagersjournal.com/
_______________________________________________
NFS maillist - [email protected]
https://lists.sourceforge.net/lists/listinfo/nfs

2004-12-06 19:21:54

by Steve Dickson

[permalink] [raw]
Subject: Re: ACL patch - is this bad or good?

Neil Brown wrote:

>So I am OK with this patch going in to nfs-utils, providing the man
>page make it clear that not all kernels support this functionality,
>and will quietly accept but ignore the flag.
>
>Anyone want to re-make the patch with a suitable change to the
>manpage?
>
>
I'll add it to my todo list....

steved.


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://productguide.itmanagersjournal.com/
_______________________________________________
NFS maillist - [email protected]
https://lists.sourceforge.net/lists/listinfo/nfs

2005-04-06 18:46:46

by Chip Salzenberg

[permalink] [raw]
Subject: Re: ACL patch - is this bad or good?

Neil Brown wrote:
>So I am OK with this patch going in to nfs-utils, providing the man
>page make it clear that not all kernels support this functionality

I've done so.
--
Chip Salzenberg - a.k.a. - <[email protected]>
Open Source is not an excuse to write fun code
then leave the actual work to others.


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
_______________________________________________
NFS maillist - [email protected]
https://lists.sourceforge.net/lists/listinfo/nfs