osmo-trx[master]: Logger: Use libosmocore logging system

Pau Espin Pedrol gerrit-no-reply at lists.osmocom.org
Fri Feb 23 11:27:40 UTC 2018


Patch Set 9:

> We might not want to break existing command line users by removing
 > the debug option completely. Is there a reasonable mapping from old
 > levels to new?

Debug option? You mean -l DEBUG? Just use "logging level all debug" in VTY. 

Regarding mapping, all the levels used are the same except this one which is re-mapped since we have less levels:

#define LOGL_ALERT LOGL_FATAL
#define LOGL_WARNING LOGL_NOTICE

Other extra levels were not used in the code anyway.

-- 
To view, visit https://gerrit.osmocom.org/6620
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I30baac89f53e927f8699d0586b43cccf88ecd493
Gerrit-PatchSet: 9
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol <pespin at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Pau Espin Pedrol <pespin at sysmocom.de>
Gerrit-Reviewer: Vadim Yanitskiy <axilirator at gmail.com>
Gerrit-HasComments: No


More information about the gerrit-log mailing list