GNU bug report logs - #13226
24.3.50; set-file-acl on MS Windows shall check ACL string format

Previous Next

Full log

Message #44 received at 13226 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Romain Francoise <romain <at> orebokech.com>
Cc: michael.albinus <at> gmx.de, monnier <at> IRO.UMontreal.CA, 13226 <at> debbugs.gnu.org
Subject: Re: bug#13226: 24.3.50;
	set-file-acl on MS Windows shall check ACL string format
Date: Wed, 19 Dec 2012 18:56:15 +0200

> From: Romain Francoise <romain <at> orebokech.com>
> Cc: Michael Albinus <michael.albinus <at> gmx.de>,  monnier <at> IRO.UMontreal.CA,  13226 <at> debbugs.gnu.org
> Date: Wed, 19 Dec 2012 17:42:18 +0100
> 
> Eli Zaretskii <eliz <at> gnu.org> writes:
> 
> > I think we are taking this feature out of proportions. It was supposed
> > to allow copying extended attributes from file to file; for that, we
> > don't need to care whether the ACL is trivial or not. But now we seem to
> > start down the road of trying to interpret the ACL, which would need
> > much more complex machinery. Do we really need that?
> 
> I agree with the sentiment, although it's not so much interpreting as just
> avoiding having to carry the ACL if it doesn't convey any interesting
> information...

Why avoid that in the first place?  It won't do any harm.

> And at least on GNU/Linux it's not a lot of code:

I don't think this is worth supporting on a single platform.  All it
does is save a single call to acl_set_file, which is not expensive
enough to avoid at the price of yet another system-dependent code
snippet.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.