[PATCH] set 'create mask' and 'directory mask' in fileserver.conf, and then eliminate fileserver.conf

[Ricky Nance]

Looks good from the end user standpoint.

Ricky

On Wed, Oct 31, 2012 at 7:41 PM, Andrew Bartlett <abartlet at samba.org> wrote:

> Attached are two patches around our default options for the AD DC.
>
> Metze,
>
> you asked me a while back about moving the rest of the default options
> from fileserver.conf into loadparm, so we wouldn't need a distinct
> fileserver.conf.
>
> The attached patch does this.
>
> Ricky and Jeremy,
>
> The other patch here follows up on your discussion, and makes 'create
> mask = 0777' and 'directory mask = 0777' the default for the AD DC.
>
> The challenge with the combination of these is that if we set things in
> loadparm, they are hard to undo - we can only determine our server role
> once we have parsed the smb.conf, and then with the current code we can
> only override the settings.  The lib/param code has a concept of a
> default flag we could consult, but it's not in the source3/param code
> right now.
>
> On the plus side, it means that the code running in 'samba-tool ntacl
> sysvolreset' will use the new defaults, which is really important.
>
> The administrator can of course set a per-share value that is different,
> and that will be honoured, as far as I understand the code.
>
> To apply the second patch, we would have to decide that this is a
> limitation we can live with, at least for now.
>
> Ricky, if you could test this, check it does what you expect for your
> specific issue and if either Metze or Jermey can review it, it would be
> good to get this into the tree.
>
> Thanks,
>
> Andrew Bartlett
> --
> Andrew Bartlett                                http://samba.org/~abartlet/
> Authentication Developer, Samba Team           http://samba.org
>
>


--