search

salortiz / LMDB_File

Perl wrapper around the OpenLDAP's LMDB
Other
8 stars 12 forks source link

undefined symbol: mdb_txn_id #25

Open salortiz opened 7 years ago

salortiz commented 7 years ago

After the v0.11 release that includes @rouzier's work some CPAN testers are failing, see: http://matrix.cpantesters.org/?dist=LMDB_File+0.11

I'm suspecting we needs to test for the availability of the symbol in lmdb library so that code can be conditionally compiled.

@rouzier, can you take a look, please?

hoytech commented 7 years ago

Is there any way to tell which liblmdb.so is being used by these tests? It would be nice to throw some diag() lines in the test before the use_ok('LMDB_File') to display this info (if possible).

I recently upgraded Alien::LMDB to the latest LMDB release, 0.9.19. This is another potential advantage of the Alien method: you can encode a minimum version LMDB dependency at the perl level (independent of what your OS packages.)

But I do agree LMDB_File should try to work with both old and new versions, and throw a run-time error if this feature wasn't compiled in.

hoytech commented 7 years ago

Although I just remembered, Alien::LMDB actually does fallback to the system verson if present... Maybe I should remove that feature, or make it opt-in...

I added an issue here:

https://github.com/hoytech/Alien-LMDB/issues/3

salortiz commented 7 years ago

I've pushed a trivial fix, I'll wait for some cpantesters.org's reports before closing this. Fingers crossed ;-)