Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Quick fix to purge-chain empty db path. #4246

Closed
marcio-diaz wants to merge 2 commits into from
Closed

Quick fix to purge-chain empty db path. #4246

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
4da5684
Fix purge-chain empty db path.
seerscode Nov 29, 2019
c588b71
Merge branch 'master' into md-fix-purge-chain
seerscode Nov 29, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion client/cli/src/lib.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -537,9 +537,15 @@ impl<'a> ParseAndPreparePurge<'a> {
G: RuntimeGenesis,
E: ChainSpecExtension,
{
let config = create_config_with_db_path::<(), _, _, _>(
let mut config = create_config_with_db_path::<(), _, _, _>(
spec_factory, &self.params.shared_params, self.version
)?;

config.database = DatabaseConfig::Path {
path: config.in_chain_config_dir(DEFAULT_DB_CONFIG_PATH).expect("We provided a base_path."),
cache_size: None,
};

Copy link
Contributor Author

@marcio-diaz marcio-diaz Nov 29, 2019 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, maybe this is wrong because of the next panic.

let db_path = match config.database {
DatabaseConfig::Path { path, .. } => path,
_ => {
Expand Down