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

Revert "Replace std::env::home_dir with dirs::home_dir (#9077)" #9097

Merged
merged 4 commits into from
Jul 12, 2018

Conversation

tomaka
Copy link
Contributor

@tomaka tomaka commented Jul 11, 2018

This reverts commit 7e77932.

Waiting for paritytech/parity-common#11
After this PR is merged, will update the Cargo.lock

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

tomaka commented Jul 11, 2018

Note that this is not necessarily the right solution. Maybe dirs should be fixed, depending on @niklasad1's opinion.

@tomaka tomaka changed the title [WIP] Revert "Replace std::env::home_dir with dirs::home_dir (#9077)" Revert "Replace std::env::home_dir with dirs::home_dir (#9077)" Jul 12, 2018
@tomaka tomaka added A0-pleasereview 🤓 Pull request needs code review. and removed A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Jul 12, 2018
@tomaka
Copy link
Contributor Author

tomaka commented Jul 12, 2018

Updated parity-common, should be ready for review.

@niklasad1 niklasad1 added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Jul 12, 2018
@debris debris merged commit acae643 into openethereum:master Jul 12, 2018
@5chdn 5chdn mentioned this pull request Jul 12, 2018
15 tasks
@tomaka tomaka deleted the revert-9077 branch July 12, 2018 12:23
5chdn pushed a commit that referenced this pull request Jul 13, 2018
…9097)

* Revert "Replace `std::env::home_dir` with `dirs::home_dir` (#9077)"

This reverts commit 7e77932.

* Restore some of the changes

* Update parity-common
5chdn added a commit that referenced this pull request Jul 17, 2018
* parity-version: betalize 2.0

* Multiple improvements to discovery ping handling (#8771)

* discovery: Only add nodes to routing table after receiving pong.

Previously the discovery algorithm would add nodes to the routing table
before confirming that the endpoint is participating in the protocol. This
now tracks in-flight pings and adds to the routing table only after receiving
a response.

* discovery: Refactor packet creation into its own function.

This function is useful inside unit tests.

* discovery: Additional testing for new add_node behavior.

* discovery: Track expiration of pings to non-yet-in-bucket nodes.

Now that we may ping nodes before adding to a k-bucket, the timeout tracking
must be separate from BucketEntry.

* discovery: Verify echo hash on pong packets.

Stores packet hash with in-flight requests and matches with pong response.

* discovery: Track timeouts on FIND_NODE requests.

* discovery: Retry failed pings with exponential backoff.

UDP packets may get dropped, so instead of immediately booting nodes that fail
to respond to a ping, retry 4 times with exponential backoff.

* !fixup Use slice instead of Vec for request_backoff.

* Add separate database directory for light client (#8927) (#9064)

* Add seperate default DB path for light client (#8927)

* Improve readability

* Revert "Replace `std::env::home_dir` with `dirs::home_dir` (#9077)" (#9097)

* Revert "Replace `std::env::home_dir` with `dirs::home_dir` (#9077)"

This reverts commit 7e77932.

* Restore some of the changes

* Update parity-common

* Offload cull to IoWorker. (#9099)

* Fix work-notify. (#9104)

* Update hidapi, fixes #7542 (#9108)

* docker: add cmake dependency (#9111)

* Update light client hardcoded headers (#9098)

* Insert Kovan hardcoded headers until #7690241

* Insert Kovan hardcoded headers until block 7690241

* Insert Ropsten hardcoded headers until #3612673

* Insert Mainnet hardcoded headers until block 5941249

* Make sure to produce full blocks. (#9115)

* Insert ETC (classic) hardcoded headers until block #6170625 (#9121)

* fix verification in ethcore-sync collect_blocks (#9135)

* Completely remove all dapps struct from rpc (#9107)

* Completely remove all dapps struct from rpc

* Remove unused pub use

* `evm bench` fix broken dependencies (#9134)

* `evm bench` use valid dependencies

Benchmarks of the `evm` used stale versions of a couple a crates that
this commit fixes!

* fix warnings

* Update snapcraft.yaml (#9132)
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