Re: Patches from fedora and fixes for LMDB backend

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


On Mon, Oct 26, 2015 at 11:21 AM, Howard Chu <hyc@xxxxxxxxxxxxxxx> wrote:
Ignacio Casal wrote:
Also, I made some fixes to the mdb backend to rename it to lmdb and to use the
renamed library lmdb instead of mdb (since it seems to have changed upstream)

Patch looks OK.

Thanks for the review.

As a plus I also modified the nmake stuff to allow to build the lmdb backend
instead of libdb. This is still not quite well tested but some comment are

 From another side, the lmdb maintainer is not interested on supporting MSVC
builds, even though I made a fork in case somebody wants to give it a try:

At this point a question arises, should we support a backend that we cannot
build on the platforms supported by cyrus-sasl? That is, should we drop lmdb
since the maintainer is not interested on it? Also we have the code that is
not building at all without the patches I pointed out.

Seems like a silly question, since LMDB builds and runs perfectly well on all platforms.

We simply choose not to support the Microsoft IDE.

It is not a silly question when the supported microsoft build for cyrus sasl is msvc :)

  -- Howard Chu
  CTO, Symas Corp. 
  Director, Highland Sun
  Chief Architect, OpenLDAP

Ignacio Casal Quinteiro

[Index of Archives]     [Info Cyrus]     [Squirrel Mail]     [Linux Media]     [Yosemite News]     [gtk]     [KDE]     [Gimp on Windows]     [Steve's Art]

  Powered by Linux