Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

compiling with new version of LMDB #7

Open
wants to merge 2 commits into
base: mdb
Choose a base branch
from
Open

Conversation

kroggen
Copy link

@kroggen kroggen commented Mar 15, 2018

Some updates in the code to make it able to be compiled against the LMDB version currently available on GitHub.

Although I am not sure about the LOCKSUFF stuff.

@kroggen
Copy link
Author

kroggen commented Mar 15, 2018

The reason I replaced the

#define MDB_USE_HASH	1

by

#ifndef _WIN32
#define MDB_USE_POSIX_SEM  1
#endif

is because of the function mdb_env_mname_init() that uses the env->me_mutexname and this one is only present when _WIN32 or MDB_USE_POSIX_SEM is defined.

An optional approach would be using some ifdefs around the mdb_env_mname_init() function and the #MUTEXNAME() on mdb.c.

@derrickgreenspan
Copy link

Thank you so much. You have saved me a lot of grief getting sqlightning working with LMDB.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants