[M] Change in libosmocore[master]: logging: add 'logging timezone (localtime|utc)'

Attention is currently required from: neels, laforge, fixeria.

pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/32043 )

Change subject: logging: add 'logging timezone (localtime|utc)' ......................................................................

Patch Set 2:

(1 comment)

File src/core/logging.c:

https://gerrit.osmocom.org/c/libosmocore/+/32043/comment/54e4c407_f9b070fa PS1, Line 874: target->timezone = timezone;

The bool vs enum aspect doesn't really affect the aspect raised by pespin, does it? […]

I don't really understand why you are even thinking about asserting or removing timestamps during logging when you can simply have a VTY command fail to set a given timestamp format when you try to set it, as per what I proposed.

I see nothing wrong with having a function returning a "rc" while others don't, specially when there's an existing case which shows that the function may fail (because system may not support it).