Samba4 patch for manipulating Unix attributes via ADUC

Gémes Géza geza at kzsdabas.hu
Thu Jul 12 23:30:24 MDT 2012


2012-07-12 15:37 keltezéssel, Andrew Bartlett írta:
> On Thu, 2012-07-12 at 23:32 +1000, Andrew Bartlett wrote:
>> On Thu, 2012-07-12 at 15:26 +0200, Gémes Géza wrote:
>>> 2012-07-12 15:15 keltezéssel, Andrew Bartlett írta:
>>>> On Thu, 2012-07-12 at 15:13 +0200, Gémes Géza wrote:
>>>>> 2012-07-12 15:04 keltezéssel, Andrew Bartlett írta:
>>>>>> On Thu, 2012-07-12 at 14:37 +0200, Gémes Géza wrote:
>>>>>>
>>>>>>> Hi,
>>>>>>>
>>>>>>> It seems, that ADUC has a built in default (both for uids and guids):
>>>>>>> 10000. After setting the value for a user/group the corresponding
>>>>>>> msSFU30MaxUidNumber/msSFU30MaxGidNumber gets filled.
>>>>>>>
>>>>>>> Shall I remove those attributes from the ldif for now?
>>>>>> Yes.
>>>>>>
>>>>>> Thanks,
>>>>>>
>>>>>> Andrew Bartlett
>>>>>>
>>>>> What about having maxuid=None and maxgid=None and if the user sets them
>>>>> at provision the attributes would get set?
>>>> Let's do that as a separate patch.  I want to think about the help text
>>>> some more.
>>>>
>>>> Thanks,
>>>>
>>>> Andrew Bartlett
>>>>
>>> Sorry,
>>>
>>> I've sent a wrong patch previously
>> This is looking good.  Just remove the maxuid/gid stuff from the
>> provision script and provision(), rather than commenting it out or
>> explicitly passing None.
> And finally, please format it as a git patch, so I can apply it.
>
> ie
> git commit
> git format-patch -1
>
> Thanks,
Hi,

Sorry for being annoying, but what is your opinion about the patch? Is 
there anything you would recommend changing?
For simplicity I've reattached the patch.

Cheers

Geza

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-When-provisioning-with-use_rfc2307-yes-populate-the-.patch
Type: text/x-patch
Size: 22175 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20120713/edaf4ddf/attachment.bin>


More information about the samba-technical mailing list