From 19db755e85b63e74ad8087dcbeee3c029f372054 Mon Sep 17 00:00:00 2001 From: Andrei Litvin Date: Fri, 5 May 2023 09:14:54 -0400 Subject: [PATCH] Replace coloredlogs usage in matter_idl with just basicConfig. (#26376) This makes the dependencies of matter_idl even smaller, generally only lark and click. Co-authored-by: Andrei Litvin --- scripts/py_matter_idl/matter_idl/lint/lint_rules_parser.py | 7 ++++--- scripts/py_matter_idl/matter_idl/matter_idl_parser.py | 7 ++++--- scripts/py_matter_idl/matter_idl/xml_parser.py | 7 ++++--- 3 files changed, 12 insertions(+), 9 deletions(-) diff --git a/scripts/py_matter_idl/matter_idl/lint/lint_rules_parser.py b/scripts/py_matter_idl/matter_idl/lint/lint_rules_parser.py index 64edb2b6850264..b8ac5d7a2b4f97 100755 --- a/scripts/py_matter_idl/matter_idl/lint/lint_rules_parser.py +++ b/scripts/py_matter_idl/matter_idl/lint/lint_rules_parser.py @@ -299,7 +299,6 @@ def CreateParser(file_name: str): # This Parser is generally not intended to be run as a stand-alone binary. # The ability to run is for debug and to print out the parsed AST. import click - import coloredlogs # Supported log levels, mapping string values required for argument # parsing into logging constants @@ -318,8 +317,10 @@ def CreateParser(file_name: str): help='Determines the verbosity of script output.') @click.argument('filename') def main(log_level, filename=None): - coloredlogs.install(level=__LOG_LEVELS__[ - log_level], fmt='%(asctime)s %(levelname)-7s %(message)s') + logging.basicConfig( + level=__LOG_LEVELS__[log_level], + format='%(asctime)s %(levelname)-7s %(message)s', + ) logging.info("Starting to parse ...") data = CreateParser(filename).parse() diff --git a/scripts/py_matter_idl/matter_idl/matter_idl_parser.py b/scripts/py_matter_idl/matter_idl/matter_idl_parser.py index 44a35fcd57e3c4..f4ba7059bf605d 100755 --- a/scripts/py_matter_idl/matter_idl/matter_idl_parser.py +++ b/scripts/py_matter_idl/matter_idl/matter_idl_parser.py @@ -541,7 +541,6 @@ def CreateParser(skip_meta: bool = False): import pprint import click - import coloredlogs # Supported log levels, mapping string values required for argument # parsing into logging constants @@ -560,8 +559,10 @@ def CreateParser(skip_meta: bool = False): help='Determines the verbosity of script output.') @click.argument('filename') def main(log_level, filename=None): - coloredlogs.install(level=__LOG_LEVELS__[ - log_level], fmt='%(asctime)s %(levelname)-7s %(message)s') + logging.basicConfig( + level=__LOG_LEVELS__[log_level], + format='%(asctime)s %(levelname)-7s %(message)s', + ) logging.info("Starting to parse ...") data = CreateParser().parse(open(filename).read(), file_name=filename) diff --git a/scripts/py_matter_idl/matter_idl/xml_parser.py b/scripts/py_matter_idl/matter_idl/xml_parser.py index e4033f98d8a92e..5a07490e9c3b68 100755 --- a/scripts/py_matter_idl/matter_idl/xml_parser.py +++ b/scripts/py_matter_idl/matter_idl/xml_parser.py @@ -33,7 +33,6 @@ import pprint import click - import coloredlogs # Supported log levels, mapping string values required for argument # parsing into logging constants @@ -59,8 +58,10 @@ help='Do not pring output data (parsed data)') @ click.argument('filenames', nargs=-1) def main(log_level, no_print, filenames): - coloredlogs.install(level=__LOG_LEVELS__[ - log_level], fmt='%(asctime)s %(levelname)-7s %(message)s') + logging.basicConfig( + level=__LOG_LEVELS__[log_level], + format='%(asctime)s %(levelname)-7s %(message)s', + ) logging.info("Starting to parse ...")