Patches to unify ACL handling in vfs_ceph + vfs_gluster

[Steve French]

I should clarify building Samba on current 4.5.1 (4.5-test branch) at
least on Fedora 24.  Seems like there are two choices:

1) configure --without-acl-support (which wouldn't help me since I was
trying to test richacl patches)
or
2) explicitly configure without ceph and without glusterfs (but with
acl support enabled)

(I had done "dnf builddep samba" so presumably had all the
dependencies, and didn't see anything obvious missing, other than
gluster and ceph which I wasn't testing). The latter is preferable for
what I was testing (richacl patches).

On Fri, Nov 4, 2016 at 2:51 PM, Steve French <smfrench at gmail.com> wrote:
> I ran into this again with master and JRAs richacl ACLs patches (and
> current 4.5-test with mine and) and default ./configure and make.  I
> had to explicitly configure "--without-acl-support"
>
> This is pretty boring current Fedora 24, but with Samba build
> dependencies "dnf builddep samba" installed
>
> 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



-- 
Thanks,

Steve