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 <msur...@sysmocom.de> Gerrit-Reviewer: Jenkins Builder (1000002) Gerrit-Reviewer: Max <msur...@sysmocom.de> Gerrit-CC: Pau Espin Pedrol <pes...@sysmocom.de> Gerrit-Comment-Date: Tue, 04 Dec 2018 12:06:05 +0000 Gerrit-HasComments: Yes Gerrit-HasLabels: No