From 6c6b940ffc80120e0334166ad03061eaf406aae9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Cl=C3=A9ment=20Pit-Claudel?= Date: Fri, 20 Sep 2024 18:49:23 +0200 Subject: [PATCH] Don't store a RichHandler object in the argparse namespace (#3394) --- pelican/__init__.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/pelican/__init__.py b/pelican/__init__.py index 96849bae..89a63e84 100644 --- a/pelican/__init__.py +++ b/pelican/__init__.py @@ -310,6 +310,9 @@ class ParseOverrides(argparse.Action): setattr(namespace, self.dest, overrides) +LOG_HANDLERS = {"plain": None, "rich": DEFAULT_LOG_HANDLER} + + def parse_arguments(argv=None): parser = argparse.ArgumentParser( description="A tool to generate a static blog, " @@ -448,7 +451,6 @@ def parse_arguments(argv=None): ), ) - LOG_HANDLERS = {"plain": None, "rich": DEFAULT_LOG_HANDLER} parser.add_argument( "--log-handler", default="rich", @@ -513,8 +515,6 @@ def parse_arguments(argv=None): if args.bind is not None and not args.listen: logger.warning("--bind without --listen has no effect") - args.log_handler = LOG_HANDLERS[args.log_handler] - return args @@ -637,7 +637,7 @@ def main(argv=None): level=args.verbosity, fatal=args.fatal, name=__name__, - handler=args.log_handler, + handler=LOG_HANDLERS[args.log_handler], logs_dedup_min_level=logs_dedup_min_level, )