Hi List,

This mail follows on the excellent and time consuming work Kamen
Mazdrashki has been provided to OpenChange NSPI server implementation
over the past months.

First of all, I truly want to apologize to Kamen for the lack of
follow-up and answers his work deserved.

Secondly, I am willing to delay openchange 0.9 release a few days so I
can merge into trunk your work.

Now the technical part ... I have started to review all the emails and
patches you have sent to the list since your initial mail on July, 3rd
2009. You'll find in current and forthcoming emails comments or svn rev
notification about all the different patches.

Let's start with the following:
=========================================================================
1. emsabp.c:emsabp_init() function uses 'ldb_wrap_connect()' to
open .ldb databases. Primary motivation for this was to be able to
handle extended syntaxes correctly (ldb_register_samba_handlers() should
be called prior opening .ldb file)

17. LDB connection fix - previously there were two connections opened -
for users.ldb and configuration.ldb. This way LDAP queries for certain
attributes didn't pass - for example 'objectCategory' attribute. Those
two connections were replaced with a connection to 'sam.ldb' and code
was refactored accordingly.
==========================================================================

I've applied a similar fix in r1575 to comply with latest samba4
requirement and the new sam.ldb + sam.ldb.d database architecture. I had
been using samdb_connect, but this is just a wrapper over
ldb_wrap_connect (furthermore generating a warning because of the
missing header declaration), I'll replace with this call in further
commit.


Regarding all the other patches, I've read the associated discussion +
gave a quick look at the associated code and at this point I must say I
agree with almost all your changes and propositions.

However, to keep the code in good shape and given that current trunk rev
slightly differs from the one you have been using, I'll apply the
changes step by step, one after the other.

I'll keep the list informed within this thread about changes/progress on
this with a generous planned number of emails ;-)

Cheers,
Julien.


Julien Kerihuel
[email protected]
OpenChange Project Manager

GPG Fingerprint: 0B55 783D A781 6329 108A  B609 7EF6 FE11 A35F 1F79


Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
devel mailing list
[email protected]
http://mailman.openchange.org/listinfo/devel

Reply via email to