pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-remsim/+/39134?usp=email )

Change subject: bankd: Add -T and -e cmdline params ......................................................................

bankd: Add -T and -e cmdline params

Change-Id: I2c33c667be77578699f4a5fb70c8b74c67f6e2cf --- M doc/manuals/chapters/remsim-bankd.adoc M src/bankd/bankd_main.c 2 files changed, 21 insertions(+), 1 deletion(-)

Approvals: Jenkins Builder: Verified osmith: Looks good to me, approved laforge: Looks good to me, but someone else must approve

diff --git a/doc/manuals/chapters/remsim-bankd.adoc b/doc/manuals/chapters/remsim-bankd.adoc index e87de52..b9db3d7 100644 --- a/doc/manuals/chapters/remsim-bankd.adoc +++ b/doc/manuals/chapters/remsim-bankd.adoc @@ -99,6 +99,16 @@ without knowledge of each other, and either of them might modify the card state (such as the currently selected file, validated PIN, etc.) in a way not expected by the other application. +*-g, --gsmtap-ip A.B.C.D*:: + Enable GSMTAP and send APDU traces to given IP. +*-G, --gsmtap-slot <0-1023>*:: + Limit tracing to given bank slot, only (default: all slots). +*-L, --disable-color*:: + Disable colors for logging to stderr. +*-T, --timestamp*:: + Prefix every log line with a timestamp. +*-e, --log-level number*:: + Set a global loglevel for all logging.

==== Examples diff --git a/src/bankd/bankd_main.c b/src/bankd/bankd_main.c index e9e4889..14c676d 100644 --- a/src/bankd/bankd_main.c +++ b/src/bankd/bankd_main.c @@ -303,6 +303,8 @@ " -g --gsmtap-ip A.B.C.D Enable GSMTAP and send APDU traces to given IP\n" " -G --gsmtap-slot <0-1023> Limit tracing to given bank slot, only (default: all slots)\n" " -L --disable-color Disable colors for logging to stderr\n" +" -T --timestamp Prefix every log line with a timestamp\n" +" -e --log-level number Set a global loglevel.\n" ); }

@@ -328,10 +330,12 @@ { "gsmtap-ip", 1, 0, 'g' }, { "gsmtap-slot", 1, 0, 'G' }, { "disable-color", 0, 0, 'L' }, + { "timestamp", 0, 0, 'T' }, + { "log-level", 1, 0, 'e' }, { 0, 0, 0, 0 } };

- c = getopt_long(argc, argv, "hVd:i:p:b:n:N:I:P:sg:G:L", long_options, &option_index); + c = getopt_long(argc, argv, "hVd:i:p:b:n:N:I:P:sg:G:LTe:", long_options, &option_index); if (c == -1) break;

@@ -380,6 +384,12 @@ case 'L': log_set_use_color(osmo_stderr_target, 0); break; + case 'T': + log_set_print_timestamp(osmo_stderr_target, 1); + break; + case 'e': + log_set_log_level(osmo_stderr_target, atoi(optarg)); + break; } } }