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

Non useable in Solaris (solved already in master, please do a release) #142

Closed
jcea opened this issue Nov 25, 2016 · 5 comments
Closed

Non useable in Solaris (solved already in master, please do a release) #142

jcea opened this issue Nov 25, 2016 · 5 comments

Comments

@jcea
Copy link
Contributor

jcea commented Nov 25, 2016

LMDB version included in py-lmdb 0.92, current release doesn't work in solaris 11/Solaris derivatives like Illumos. The fix is already committed in master LMDB and, in fact, it is already included in master py-lmdb: https://github.com/dw/py-lmdb/blob/master/lib/mdb.master/mdb.c#L4997

Can I request a new release containing that fix?. Thanks!.

More details:

https://github.com/joyent/pkgsrc/issues/431
LMDB/lmdb@c367c1f

@dw
Copy link
Collaborator

dw commented Nov 25, 2016

Hi @jcea,

The recent changes in py-lmdb to support the master branch are not ready yet, due to linking issues on Windows I haven't had time to straighten out yet. I will hopefully get a chance to sort it out this weekend, after which a new "lmdb_master" Python package will exist that installs the master branch.

@jcea
Copy link
Contributor Author

jcea commented Dec 15, 2016

Any news? :-)

@dw
Copy link
Collaborator

dw commented Dec 15, 2016

Sorry dude,

The lack of sunlight these past 2 months has severely impacted my productivity. There is stuff on master, but it is too complicated for all it is supposed to be doing, feel free to work at it.

The remaining problem is the linking to the kernel APIs doesn't work properly for 32bit. Upstream are chatting about replacing those static links with GetProcAddress() but I don't think anything is on mdb.master yet.

Will pick this up when I have the energy, but I really don't know when that will be, sorry.

@dw
Copy link
Collaborator

dw commented Dec 15, 2016

Looks like nobody has done anything to mdb.master yet. The GetProcAddress stuff is actually a really easy change, I can rip out most of the crap on py-lmdb master once it's done. If you're feeling adventurous and know a bit of C, maybe submit a patch to OpenLDAP ITS instead :)

@jnwatson
Copy link
Owner

py-lmdb has been updated.

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

No branches or pull requests

3 participants