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

deps: LevelDB 1.22 #12

Closed
wants to merge 5 commits into from
Closed

deps: LevelDB 1.22 #12

wants to merge 5 commits into from

Conversation

tynes
Copy link
Member

@tynes tynes commented May 11, 2020

This Pull Request bumps the vendored version of LevelDB to 1.22.

I have tested it on both Windows and Linux. It builds and the included test suite passes on both systems.

There are a lot of changed files but the most important one to look at is https://github.com/tynes/bdb/blob/leveldb-1.22/deps/leveldb/leveldb.gyp

Some of the build variables have changed, I think that I removed the dead ones and added the required new ones. I directly copied the leveldb directory from the sources at https://github.com/google/leveldb/releases/tag/1.22

@braydonf
Copy link
Contributor

Any progress with updating upstream leveldown? As there are very few tests for bdb, that will probably be necessary to get more complete testing.

@tynes tynes closed this Jul 28, 2020
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.

2 participants