On Tuesday 10 January 2012 09:56:02 Pierangelo Masarati wrote:

...

Now, if ldapdb is incompatible with slapd at a fundamental level, and causes problems when loaded, I think not to load it is the right thing to do, and this is what my patch does.

An alternative solution consists in creating a specific folder containing plugins one wants to load, and start slapd with SASL_PATH set to point to that folder. This allows fine tuning of what to load without modifying a single line of code.

That is certainly true, however, it misses the point I am trying to make. I want to provide a fix for the problem I encountered.

From a user's perspective, it is certainly preferable for the whole thing

to 'just work', instead of being provided with instructions about how to configure one's system to avoid problems that arise when using the standard configuration. This places the burden on the user, for no other gain than being able to load a module that cannot be used anyway. With the change I proposed, slapd can work with every standard cyrus-sasl installation right away, no special configuration required, and the problem is solved once and forever.

Thomas