From 36d392e9b198a1b82817b7db17c8f00f3189db3d Mon Sep 17 00:00:00 2001 From: Sebastian Spaeth Date: Tue, 8 Nov 2022 18:55:23 +0100 Subject: [PATCH] Better output on dryrun, make clear what we would delete now --- cleanmedia | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/cleanmedia b/cleanmedia index cc3d2fe..e97e29e 100755 --- a/cleanmedia +++ b/cleanmedia @@ -151,7 +151,7 @@ def parse_options(): logging.basicConfig(level=loglevel, format= '%(levelname)s - %(message)s') return args - +#----------------------------------------------------------------------- if __name__ == '__main__': args = parse_options() (MEDIA_PATH, CONN_STR) = read_config(args.config) @@ -160,7 +160,10 @@ if __name__ == '__main__': files = mr.get_remote_media() for file in files: if file.create_date < cleantime: - if not file.exists(): - logging.info(f"file id {file.media_id} does not physically exist (path {file.fullpath})") - if not args.dryrun: + if args.dryrun: # the great pretender + if file.exists(): + logging.info(f"Pretending to delete file id {file.media_id} on path {file.fullpath}.") + else: + logging.info(f"file id {file.media_id} does not physically exist (path {file.fullpath})") + else: file.delete()