Re: [Acl-devel] Man page patch for setfacl

[Mike Frysinger]

On 22 Jan 2018 21:14, Andreas Grünbacher wrote:
> 2018-01-22 21:02 GMT+01:00 Mike Frysinger <address@hidden>:
> > On 22 Jan 2018 17:36, Andreas Grünbacher wrote:
> >> 2018-01-22 17:16 GMT+01:00 Michael Orlitzky <address@hidden>:
> >> > On 01/22/2018 10:48 AM, Andreas Grünbacher wrote:
> >> >> If you create all-numeric user names, you deserve all the troubles
> >> >> you've been asking for.
> >> >
> >> > Do I still deserve to suffer if I'm trying to fix a mistake that someone
> >> > else made 15 years ago and that I've been called in to fix?
> >>
> >> So if change the behavior of setfacl to match current coreutils,
> >> should getfacl be changed to prefix numeric IDs with + signs as well?
> >> I doubt it -- setfacl would be happy, but other tools might break. So
> >> then that needs to be documented as well.
> >
> > i think we should change the behavior to match coreutils wrt looking up
> > the account first (like my patch does).  however, i'm not exactly keen
> > on the + hack since that too is a valid character in a username.  maybe
> > we should introduce a new commandline option like --numeric that'll force
> > all accounts to be numeric and do no lookups ?
> 
> Let's not add a command-line option, it's just not useful enough.

so you want to include support for the + or just use my patch as-is ?
-mike

signature.asc
Description: Digital signature