Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

use kvdb-* and parity-snappy crates from crates.io #9441

Merged
merged 2 commits into from
Sep 3, 2018

Conversation

andresilva
Copy link
Contributor

This should also fix the RocksDB build issues on ARM and Windows since it includes (paritytech/rust-rocksdb#19). I want to run a CI job on ARM and Windows to make sure it is passing so I'll label this as inprogress until that's done.

@andresilva andresilva added A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. M4-core ⛓ Core client code / Rust. labels Aug 30, 2018
@5chdn 5chdn added this to the 2.1 milestone Aug 30, 2018
@andresilva
Copy link
Contributor Author

@5chdn
Copy link
Contributor

5chdn commented Aug 30, 2018

Thanks for taking care of this!

@andresilva andresilva added A0-pleasereview 🤓 Pull request needs code review. and removed A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Sep 3, 2018
@andresilva
Copy link
Contributor Author

Everything is green again: https://gitlab.parity.io/parity/parity-ethereum/pipelines/22967

@5chdn 5chdn added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Sep 3, 2018
@andresilva andresilva merged commit c0eb30b into master Sep 3, 2018
@5chdn 5chdn deleted the andre/fix-rocksdb-build branch September 3, 2018 22:22
dvdplm added a commit that referenced this pull request Sep 4, 2018
* master:
  evmbin: escape newlines in json errors (#9458)
  use kvdb-* and parity-snappy crates from crates.io (#9441)
  Add EIP-1014 transition config flag (#9268)
  add tags for runner selection of build-linux jobs (#9451)
  Remove unused BlockStatus::Pending (#9447)
  ci: only include local paths in coverage script (except target) (#9437)
  Add POA Networks: Core and Sokol (#9413)
  docker: install missing dependencies in arm target dockerfiles (#9436)
  Random small cleanups (#9423)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants