commit cfc312487745cf7ce73c26b6fe2976d80eabf501
author Raef Coles <raef.coles@arm.com> Fri Apr 04 09:38:47 2025 +0100
committer Antonio de Angelis <Antonio.deAngelis@arm.com> Wed Apr 09 12:27:08 2025 +0200
tree d33b59b7a66e794f039b06dfc8a6cd6ecd46c0c4
parent e5cd7482bbbc043355fda299b0f6bdb4d09fc0e1

Tools: Fix script logging

Fix issue preventing scripts respecting --log_level argument, by adding
a non-default handler. Also, add script name to log output.

Change-Id: I100a5c29276d362dc38cd6c46f472e5991873293
Signed-off-by: Raef Coles <raef.coles@arm.com>

tools/modules/c_struct.py

diff --git a/tools/modules/c_struct.py b/tools/modules/c_struct.py
index 08c0428..e88bebc 100644
--- a/tools/modules/c_struct.py
+++ b/tools/modules/c_struct.py
@@ -12,7 +12,7 @@
 import os
 
 import logging
-logger = logging.getLogger("TF-M")
+logger = logging.getLogger("TF-M.{}".format(__name__))
 
 from rich import inspect
 
@@ -643,7 +643,8 @@
     parser.add_argument("--c_file_to_mirror_includes_from", help="name of the c file to take", required=True)
     parser.add_argument("--log_level", help="log level", required=False, default="ERROR", choices=logging._levelToName.values())
     args = parser.parse_args()
-    logger.setLevel(args.log_level)
+    logging.getLogger("TF-M").setLevel(args.log_level)
+    logger.addHandler(logging.StreamHandler())
 
     includes = c_include.get_includes(args.compile_commands_file, args.c_file_to_mirror_includes_from)
     defines = c_include.get_defines(args.compile_commands_file, args.c_file_to_mirror_includes_from)