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

replace_home for password_files, reserved_peers and log_file #8324

Merged
merged 2 commits into from
Apr 9, 2018

Conversation

sorpaas
Copy link
Collaborator

@sorpaas sorpaas commented Apr 6, 2018

Fixes #7392

This adds support for $HOME and $BASE variable in account's password_files, network's reserved_peers and log's log_file. Looks that those three are the only remaining paths that don't yet support replace_home.

@sorpaas sorpaas added the A0-pleasereview 🤓 Pull request needs code review. label Apr 6, 2018
@sorpaas sorpaas added this to the 1.11 milestone Apr 6, 2018
@sorpaas sorpaas added the M4-core ⛓ Core client code / Rust. label Apr 6, 2018
@5chdn 5chdn added A1-onice 🌨 Pull request is reviewed well, but should not yet be merged. and removed A1-onice 🌨 Pull request is reviewed well, but should not yet be merged. labels Apr 9, 2018
@debris debris added A8-looksgood 🦄 Pull request is reviewed well. B0-patchthis and removed A0-pleasereview 🤓 Pull request needs code review. labels Apr 9, 2018
@debris debris merged commit 431b27d into openethereum:master Apr 9, 2018
@tomusdrw tomusdrw mentioned this pull request Apr 10, 2018
tomusdrw pushed a commit that referenced this pull request Apr 10, 2018
* replace_home for password_files, reserved_peers and log_file

* typo: arg_log_file is Option
tomusdrw pushed a commit that referenced this pull request Apr 10, 2018
* replace_home for password_files, reserved_peers and log_file

* typo: arg_log_file is Option
@tomusdrw tomusdrw mentioned this pull request Apr 10, 2018
5chdn pushed a commit that referenced this pull request Apr 10, 2018
* Warp-only sync with warp-barrier [blocknumber] flag. (#8228)

* Warp-only sync with warp-after [blocknumber] flag.

* Fix tests.

* Fix configuration tests.

* Rename to warp barrier.

* Allow unsafe js eval on Parity Wallet. (#8204)

* Update musicoin spec in line with gmc v2.6.2 (#8242)

* Supress TemporaryInvalid verification failures. (#8256)

* Include suicided accounts in state diff (#8297)

* Include suicided accounts in state diff

* Shorten form match -> if let

* Test suicide trace diff in State

* replace_home for password_files, reserved_peers and log_file (#8324)

* replace_home for password_files, reserved_peers and log_file

* typo: arg_log_file is Option

* Enable UI by default, but only display info page.

* Fix test.

* Fix naming and remove old todo.

* Change "wallet" with "browser UI"
5chdn pushed a commit that referenced this pull request Apr 10, 2018
* Update musicoin spec in line with gmc v2.6.2 (#8242)

* Supress TemporaryInvalid verification failures. (#8256)

* Include suicided accounts in state diff (#8297)

* Include suicided accounts in state diff

* Shorten form match -> if let

* Test suicide trace diff in State

* replace_home for password_files, reserved_peers and log_file (#8324)

* replace_home for password_files, reserved_peers and log_file

* typo: arg_log_file is Option

* Bump version in util/version
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.

3 participants