Skip to content

Conversation

@Fuzzbawls
Copy link
Collaborator

Switch to a git subtree for leveldb (similar to univalue and secp256k1)

This also fixes #534

@Mrs-X
Copy link

Mrs-X commented Jul 28, 2018

Since this is a huge PR, would it be sufficient to run a Gitian build for all OSs to test it properly?

@Fuzzbawls
Copy link
Collaborator Author

build and unit tests are always good, but this one also needs extensive runtime testing with new and existing chain/wallet data.

Copy link

@Warrows Warrows left a 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.

@Mrs-X
Copy link

Mrs-X commented Aug 17, 2018

I'm starting to test this right now and will report back in a day or so...

@Mrs-X
Copy link

Mrs-X commented Aug 17, 2018

  • First result: ACK that it fixes Mac 10.13 warning: 'OSMemoryBarrier' is deprecated #534 There's still a million Qt-warnings, though. At least for Qt 5.4. But it's not our code.
  • Gitian-build: ACK
  • Qt-wallet (Linux) with existing chain/wallet data: ACK
  • Qt-wallet (OSX) with existing chain/wallet data: ACK

(to be continued...)

Copy link

@Mrs-X Mrs-X left a 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...

@Mrs-X Mrs-X merged commit 85c2a28 into PIVX-Project:master Aug 18, 2018
Mrs-X added a commit that referenced this pull request Aug 18, 2018
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
@ghost ghost removed the review label Aug 18, 2018
@Fuzzbawls Fuzzbawls deleted the 2018_leveldb-update branch January 10, 2020 00:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Mac 10.13 warning: 'OSMemoryBarrier' is deprecated

3 participants