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

[Andrew Bartlett]

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-s3-param-Move-the-options-needed-for-running-smbd-in.patch
Type: text/x-patch
Size: 4836 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20121101/e68989c3/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-file_server-put-set-create-mask-and-directory-mask-i.patch
Type: text/x-patch
Size: 1055 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20121101/e68989c3/attachment-0001.bin>