new idmap code

Gerald (Jerry) Carter jerry at samba.org
Mon Dec 11 17:02:20 GMT 2006


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

simo wrote:

> Let me explain why I used this approach:
> 1. I tried to be as less invasive as possible and parametric 
> options are perfect for that.
> 2. I wanted to give backends a way to be able to have 
> their own parameters, without resorting to overloading
> existing parameters as we did in the past. The current
> code allows foobar backend to have it's own
> foo and bar options without the need to change 
> anything in loadparm.c or idmap.c

Like I said, i can't think of a good alternative.  However,
we will need to finally statr providing individual man pages
for the various plugins.  We should have been doing this for
the VFS modules already.

>> Second, the gid_to_sid() code is failing for the when using
>> the nss idmap backend.
>
> Ok, I am sure we can find a way to handle this. I'd like 
> to do it after the merge if you are ok with that.

Sure.  The group sid2gid code was broken in this case before.
It's just more broken without that change I mentioned.

> Ok, do you want me to apply this big patch?
> http://samba.org/~idra/patches/samba3_new_idmap.patch
> 
> Or would you prefer to see the thing committed in 
> pieces somehow?

One patch is fine with me.   Thanks.





cheers, jerry
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.3 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFFfY8cIR7qMdg1EfYRAlMYAKCSjqb7nPuzzMNNEi2eRkWOcYa5VgCfY2CM
/iJAaa5KP9SA3zel9+1CKZ0=
=OYNs
-----END PGP SIGNATURE-----


More information about the samba-technical mailing list