Re: ITS#6055

[masarati@aero.polimi.it]

> On Tue, 2010-04-20 at 15:37 +0200, masarati@aero.polimi.it wrote:
>> > That's no problem at all.
>>
>> Good.  This version explicitly check for instantiation as global, and
>> also
>> does some further sanity checks prior to startup.
>>
>> <ftp://ftp.openldap.org/incoming/pierangelo-masarati-2010-04-20-rdnval.2.c>
>>
>> > It now just fails in the same was as without
>> > the patch.  I'll mention it on openldap-technical if I can't get
>> > assistance on #ldap.
>>
>> Not quite clear what patch and what failure you refer to.  Anyway, I've
>> tested this version of the overlay as global (refuses to start) and on a
>> database, either empty or populated, and it works as intended.  Please
>> test and report.  Thanks, p.
>
> It seems to be working well.  When can I expect it to be in the tree?
>
> (I'll then require folks to use the current OpenLDAP CVS if they want to
> use the LDAP backend).

I can commit it in a minute.  It's not 100% clear to me what's the most
appropriate location: slapd/overlays/ may not, as this looks like
something definitely specific to Samba4; probably, contrib/slapd-modules/
is more appropriate.  Perhaps, a contrib/slapd-modules/samba4/ would be
the most appropriate, along with future specific (i.e. not of general
usefulness) modules, as we did for the nssov module.

Comments?  I mean, from others, not only from you :)

p.