Skip to content

Commit

Permalink
ledger-tool: Fix create-snapshot default value for output_directory (#…
Browse files Browse the repository at this point in the history
…3148)

The arguments to specify full and incremental snapshot archives paths
used to be a global argument; these were moved to only be instantiated
on commands that needed them in #1773.

But, when the arguments were moved from app-level to subcommand-level,
the code that matches the arguments was not updated to look at
subcommand-matches instead of app-matches.

(cherry picked from commit 1d9947c)
  • Loading branch information
steviez authored and mergify[bot] committed Oct 13, 2024
1 parent 0b64361 commit 8eae852
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions ledger-tool/src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1722,11 +1722,11 @@ fn main() {
let is_minimized = arg_matches.is_present("minimized");
let output_directory = value_t!(arg_matches, "output_directory", PathBuf)
.unwrap_or_else(|_| {
let snapshot_archive_path = value_t!(matches, "snapshots", String)
let snapshot_archive_path = value_t!(arg_matches, "snapshots", String)
.ok()
.map(PathBuf::from);
let incremental_snapshot_archive_path =
value_t!(matches, "incremental_snapshot_archive_path", String)
value_t!(arg_matches, "incremental_snapshot_archive_path", String)
.ok()
.map(PathBuf::from);
match (
Expand Down

0 comments on commit 8eae852

Please sign in to comment.