Change in libosmocore[master]: Log: add stdout target

[Max]

Max has posted comments on this change. ( https://gerrit.osmocom.org/12093 )

Change subject: Log: add stdout target
......................................................................


Patch Set 1:

(2 comments)

As for rename - I can change it in the next revision.

https://gerrit.osmocom.org/#/c/12093/1/src/logging.c
File src/logging.c:

https://gerrit.osmocom.org/#/c/12093/1/src/logging.c@926
PS1, Line 926: 		if (target->tgt_file.out != stdout)
> This looks wrong or at least really messy.
Could you elaborate? How else should it look like? And why you think it's wrong?


https://gerrit.osmocom.org/#/c/12093/1/src/vty/logging_vty.c
File src/vty/logging_vty.c:

https://gerrit.osmocom.org/#/c/12093/1/src/vty/logging_vty.c@881
PS1, Line 881: 		/* we don't support vty logging to stdout to avoid messing up vty prompt and user input */
> I don't get what's the issue here.
Vty code prints to stdout. If we log to stdout as well, than user will see vty prompts interspersed with log output which looks really messy. That's why I'm reluctant to add stdout target to vty.



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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia786361f5f687e43b27d87a45b4630bca58bcfe8
Gerrit-Change-Number: 12093
Gerrit-PatchSet: 1
Gerrit-Owner: Max <msuraev at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Max <msuraev at sysmocom.de>
Gerrit-CC: Pau Espin Pedrol <pespin at sysmocom.de>
Gerrit-Comment-Date: Tue, 04 Dec 2018 12:06:05 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20181204/f194b796/attachment.htm>