-
Notifications
You must be signed in to change notification settings - Fork 724
[Build] Update to latest leveldb #671
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
Conversation
git-subtree-dir: src/leveldb git-subtree-split: 64052c7
(cherry picked from commit ffee03c)
Also for libsecp256k1 now that it has been updated
|
Since this is a huge PR, would it be sufficient to run a Gitian build for all OSs to test it properly? |
|
build and unit tests are always good, but this one also needs extensive runtime testing with new and existing chain/wallet data. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK. About testing, I've been running it on a Windows 10 (64 bits) staking wallet for 48+ hours now. Opening and closing it occasionally.
|
I'm starting to test this right now and will report back in a day or so... |
(to be continued...) |
Mrs-X
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- 24h staking (Linux): ACK
Merging...
85c2a28 travis: run subtree checks for leveldb (Fuzzbawls) 1d99c21 build: update build system for new leveldb (Fuzzbawls) 4f5b53d Squashed 'src/leveldb/' content from commit 64052c7 (Fuzzbawls) d3e48bd Remove leveldb path in prep for subtree init (Fuzzbawls) Tree-SHA512: de3d6fc36393866c8aefde550bb015e6de1c77684653d2e140ff90d638aa3d47c63b12f826fa474cc004f888d76c4108c4516f99cb6c77afb40336ea8f631cfa
Switch to a git subtree for leveldb (similar to univalue and secp256k1)
This also fixes #534