abee79f vfs: Use posix_sys_acl_blob_get_file in vfs_gpfs for posix ACLs

simo idra at samba.org
Thu Oct 11 10:50:16 MDT 2012


On Thu, 2012-10-11 at 09:37 -0700, Jeremy Allison wrote:
> On Thu, Oct 11, 2012 at 12:32:12PM -0400, simo wrote:
> > 
> > Given the above 'incident' is not an isolated episode but happens
> > frequently, I say +1
> > I already generally get a sign-off from reviewers and let them push
> > anyway.
> 
> This should not be about what you do, or what Andrew does.
> 
> This should be a Team decision made to help improve the
> quality of the code overall, and that's what I'm proposing.

I agree, sorry for ranting on this specific episode, I agree on the
proposal in general.

> If it helps people - this is exactly what we do here @ Google.
> 
> I know we don't have Google's resources, but we should note
> best practices in the area.

Just for the record, in all my other projects @ Red Hat (big and small
except for one-man ones) we use a review before commit approach that
works quite well too.

Simo.

-- 
Simo Sorce
Samba Team GPL Compliance Officer <simo at samba.org>
Principal Software Engineer at Red Hat, Inc. <simo at redhat.com>



More information about the samba-technical mailing list