On Sun, May 20, 2018 at 07:43:41PM -0700, Paul B. Henson wrote:
> This patch updates the openldap port to use modules instead of being
> monolithic. I've been using it in production for six months or so and
> haven't had any issues.

It needs a bit more than that:
- the versionning of the modules feels 'wrong', and should be fixed
  within the build system. The fact that you have to symlink them is a
red herring.. imo, we should just have .so files with or without the
-2.4 suffix, depending whether we plan to have the same for 2.3 and the
server is able to find them (ie look at the code listing the modules)

- this definitely needs a current.html entry to warn people of the
  configuration changes, because in the current state afair all the
modules are bundled/available, and your proposition would force people
to load the modules they actually use.

Reply via email to