[illumos-Developer] webrev for #948: chmod(1M) is not -R friendly when doing ACLs

Yuri Pankov yuri.pankov at gmail.com
Fri May 6 19:34:31 PDT 2011


On Fri, May 06, 2011 at 10:30:10PM -0400, Albert Lee wrote:
> Ditto. Has Gordon asked you about fixing the write_set/modify_set ACLs in chmod?

No.. And I minimized the changes in this webrev (had some error messages
improvements and a bit of code cleanup, will do it separately) for this
to be accepted :-)


Yuri

> On Fri, May 6, 2011 at 12:47 AM, Garrett D'Amore <garrett at damore.org> wrote:
> > I think this looks ok.
> >
> >        - Garrett
> >
> > On Fri, 2011-05-06 at 03:20 +0400, Yuri Pankov wrote:
> >> On Thu, Apr 28, 2011 at 05:52:15AM +0400, Yuri Pankov wrote:
> >> > Hi,
> >> >
> >> > This is webrev for #948: chmod(1M) is not -R friendly when doing ACLs.
> >>
> >> To get this done, here's a new webrev just making the errors in doacl()
> >> non-fatal to make recursive chmod continue if we can't modify one of the
> >> files/dirs (common.c changed to appease cstyle).
> >>
> >> https://www.xvoid.org/illumos/webrev/948-chmod-r-acl/
> >>
> >>
> >> Yuri



More information about the Developer mailing list