Patches to unify ACL handling in vfs_ceph + vfs_gluster

[Ira Cooper]

Pushed.

-Ira

On Fri, Jul 29, 2016 at 12:33 AM, Steve French <smfrench at gmail.com> wrote:

> Reviewed-and-Tested-by: Steve French <sfrench at samba.org>
>
> On Tue, Jul 26, 2016 at 12:38 PM, Ira Cooper <ira at wakeful.net> wrote:
> > Try this Steve.  It should fix the issue.
> >
> > Cheers,
> >
> > -Ira
> >
> > On Tue, Jul 26, 2016 at 2:22 AM, Steve French <smfrench at gmail.com>
> wrote:
> >>
> >> Samba master is broken due to recent ACL changes, even if you build
> >> with "--without-acl-support" it tries to pull in 'acl' for
> >> POSIXACL_XATTR
> >>
> >> On Tue, Jul 26, 2016 at 1:08 AM, Steve French <smfrench at gmail.com>
> wrote:
> >> > With master updated with the recent ACL changes in the last five days,
> >> > I noticed a build break on master (our builds have richacl enabled,
> >> > but don't include posix acl).  Even on RHEL7 this will fail
> >> > (libacl-devel is installed, but acls aren't enabled).  See excerpts
> >> > from configure tests relating to ACLs below:
> >> >
> >> > 23:43:35 Checking for header acl/libacl.h
>  :
> >> > yes
> >> > 23:43:36 Checking for header sys/acl.h
> :
> >> > yes
> >> > 23:44:30 Checking for _acl
> >> >                                   : not found
> >> > 23:44:30 Checking for __acl
> >> >                                   : not found
> >> > 23:44:30 Checking for chmod
> >> >                                   : ok
> >> > 23:44:31 Checking for _facl
> >> >                                   : not found
> >> > 23:44:31 Checking for __facl
> >> >                                   : not found
> >> > 23:44:36 building on linux2
> >> > 23:44:36 Checking for header sys/richacl.h
> >> >                                   : yes
> >> > 23:44:36 Checking for library richacl
> >> >                                   : yes
> >> > 23:44:36 Checking for richacl_get_file
> >> >                                   : ok
> >> > 23:44:36 Using Linux RICHACLs
> >> > ...
> >> >
> >> > 23:44:49 Checking project rules ...
> >> > 23:44:49 Unknown dependency 'acl' in 'POSIXACL_XATTR'
> >> >
> >> > On Fri, Jul 22, 2016 at 5:20 PM, Jeremy Allison <jra at samba.org>
> wrote:
> >> >> On Thu, Jul 21, 2016 at 08:47:07PM -0400, Ira Cooper wrote:
> >> >>> Forgot to git commit --amend that one :/.
> >> >>>
> >> >>> I have the exact same fix locally.
> >> >>>
> >> >>> Just passed my local build.
> >> >>
> >> >> LGTM. Pushed !
> >> >>
> >> >
> >> >
> >> >
> >> > --
> >> > Thanks,
> >> >
> >> > Steve
> >>
> >>
> >>
> >> --
> >> Thanks,
> >>
> >> Steve
> >
> >
>
>
>
> --
> Thanks,
>
> Steve
>