From defa4dfa9b00600ddb912e50f4404539fba9b0ad Mon Sep 17 00:00:00 2001 From: Thomas Waldmann Date: Tue, 13 Oct 2020 22:25:44 +0200 Subject: [PATCH] fix --dry-run and --stats coexistence, fixes #5415 --- src/borg/archiver.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/borg/archiver.py b/src/borg/archiver.py index de2e18305b..f95b8cdfa1 100644 --- a/src/borg/archiver.py +++ b/src/borg/archiver.py @@ -3389,10 +3389,11 @@ def define_archive_filters_group(subparser, *, sort_by=True, first_last=True): help='create backup') subparser.set_defaults(func=self.do_create) - dryrun_group = subparser.add_mutually_exclusive_group() - dryrun_group.add_argument('-n', '--dry-run', dest='dry_run', action='store_true', + # note: --dry-run and --stats are mutually exclusive, but we do not want to abort when + # parsing, but rather proceed with the dry-run, but without stats (see run() method). + subparser.add_argument('-n', '--dry-run', dest='dry_run', action='store_true', help='do not create a backup archive') - dryrun_group.add_argument('-s', '--stats', dest='stats', action='store_true', + subparser.add_argument('-s', '--stats', dest='stats', action='store_true', help='print statistics for the created archive') subparser.add_argument('--list', dest='output_list', action='store_true',