Change in osmocom-bb[master]: trx_toolkit/app_common: add options to enable time printing

[fixeria]

fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmocom-bb/+/17760 )


Change subject: trx_toolkit/app_common: add options to enable time printing
......................................................................

trx_toolkit/app_common: add options to enable time printing

Change-Id: Ie5d14a261e17af554f7132b03d58549a4831dcdb
---
M src/target/trx_toolkit/app_common.py
1 file changed, 25 insertions(+), 5 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/60/17760/1

diff --git a/src/target/trx_toolkit/app_common.py b/src/target/trx_toolkit/app_common.py
index a8d6c8d..6e53b80 100644
--- a/src/target/trx_toolkit/app_common.py
+++ b/src/target/trx_toolkit/app_common.py
@@ -4,7 +4,7 @@
 # TRX Toolkit
 # Common helpers for applications
 #
-# (C) 2018 by Vadim Yanitskiy <axilirator at gmail.com>
+# (C) 2018-2020 by Vadim Yanitskiy <axilirator at gmail.com>
 #
 # All Rights Reserved
 #
@@ -29,6 +29,9 @@
 	# Example: [DEBUG] ctrl_if_bts.py:71 Recv POWEROFF cmd
 	LOG_FMT_DEFAULT = "[%(levelname)s] %(filename)s:%(lineno)d %(message)s"
 
+	# Default time / date format (e.g. 2003-01-23 00:29:50)
+	LOG_TIME_FMT_DEFAULT = "%Y-%m-%d %H:%M:%S"
+
 	def app_print_copyright(self, holders = []):
 		# Print copyright holders if any
 		for date, author in holders:
@@ -40,8 +43,9 @@
 			"This is free software: you are free to change and redistribute it.\n" \
 			"There is NO WARRANTY, to the extent permitted by law.\n")
 
-	def add_log_handler(self, lh, log_level, log_fmt):
-		lf = log.Formatter(log_fmt)
+	def add_log_handler(self, lh, log_level, log_fmt, time_fmt, log_time = False):
+		log_fmt = "%(asctime)s " + log_fmt if log_time else log_fmt
+		lf = log.Formatter(log_fmt, time_fmt)
 		ll = log.getLevelName(log_level)
 
 		log.root.addHandler(lh)
@@ -51,12 +55,14 @@
 	def app_init_logging(self, argv):
 		# Default logging handler (stderr)
 		self.add_log_handler(log.StreamHandler(), argv.log_level,
-				     argv.log_fmt)
+				     argv.log_fmt, argv.log_time_fmt,
+				     argv.log_time)
 
 		# Optional file handler
 		if argv.log_file_name is not None:
 			self.add_log_handler(log.FileHandler(argv.log_file_name), argv.log_file_level,
-					     argv.log_file_fmt)
+					     argv.log_file_fmt, argv.log_file_time_fmt,
+					     argv.log_file_time)
 
 		# Set DEBUG for the root logger
 		log.root.setLevel(log.DEBUG)
@@ -66,6 +72,13 @@
 			dest = "log_level", type = str, default = "DEBUG",
 			choices = ["DEBUG", "INFO", "WARNING", "ERROR", "CRITICAL"],
 			help = "Set logging level (default %(default)s)")
+		parser.add_argument("--log-time",
+			dest = "log_time", action = "store_true",
+			help = "Prefix each log message with the current time")
+		parser.add_argument("--log-time-format", metavar = "FMT",
+			dest = "log_time_fmt", type = str,
+			default = self.LOG_TIME_FMT_DEFAULT,
+			help = "Set time format (default %(default)s)")
 		parser.add_argument("--log-format", metavar = "FMT",
 			dest = "log_fmt", type = str, default = self.LOG_FMT_DEFAULT,
 			help = "Set logging message format")
@@ -77,6 +90,13 @@
 			dest = "log_file_level", type = str, default = "DEBUG",
 			choices = ["DEBUG", "INFO", "WARNING", "ERROR", "CRITICAL"],
 			help = "Set logging level for file (default %(default)s)")
+		parser.add_argument("--log-file-time",
+			dest = "log_file_time", action = "store_true",
+			help = "Prefix each log message with the current time")
+		parser.add_argument("--log-file-time-format", metavar = "FMT",
+			dest = "log_file_time_fmt", type = str,
+			default = self.LOG_TIME_FMT_DEFAULT,
+			help = "Set time format for file (default %(default)s)")
 		parser.add_argument("--log-file-format", metavar = "FMT",
 			dest = "log_file_fmt", type = str, default = self.LOG_FMT_DEFAULT,
 			help = "Set logging message format for file")

-- 
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/17760
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: Ie5d14a261e17af554f7132b03d58549a4831dcdb
Gerrit-Change-Number: 17760
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <axilirator at gmail.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20200408/d5557f33/attachment.htm>