basho / eleveldb

Erlang LevelDB API
263 stars 176 forks source link

Feature/use system snappy - don't merge - needs more work #244

Closed bryanhuntesl closed 5 years ago

bryanhuntesl commented 6 years ago

Use system snappy lib

We were shipping an old version of snappy, storing it as a tar file, extracting, building the lib and then staticly compiling it into our eleveldb.so.

Why do this? Snappy is a library (snappy) via brew on OSX, via apt-get/yum (libsnappy-dev).

Our version is also old, and reduces farm to plate traceability.

This changeset removes the bundled tar file (c_src/snappy-1.0.4.tar.gz) and uses the system lib instead.

Licenser commented 6 years ago

👍 works on SmartOS

bryanhuntesl commented 5 years ago

Superceeded by #254