idmap redesign breaks existing configs

[simo]

On Tue, 2007-02-06 at 09:15 -0600, Gerald (Jerry) Carter wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> Simo,
> 
> The idmap changes are not backwards compatible with a
> simple config like:
> 
> ldap admin dn         = "cn=smbadmin,ou=people,dc=plainjoe,dc=org"
> ldap suffix           = dc=plainjoe,dc=org
> ldap idmap suffix     = ou=idmap
> idmap backend = ldap:ldap://ldap.plainjoe.org/
> idmap uid = 60000-120000
> idmap gid = 60000-120000
> 
> There are two isssues.  (a) the idmap_init parsing for
> the compat backends does separate the backend from the
> parameters (i've fixed this locally), and (b) the
> idmap_ldap_db_init() is looking for the new config options
> and therefore fails to initialize.

I am sorry about that, I had the parameters parsing done right at some
point I am sure, I must have been distracted later in the process :-(

> I hate to say it but upgrades were obviously not tested
> in this case. I'm working on a fix now.

Yeah I didn't test the upgrade in the latest phases, my bad.

Simo.

-- 
Simo Sorce
Samba Team GPL Compliance Officer
email: idra at samba.org
http://samba.org