<p style="white-space: pre-wrap; word-wrap: break-word;">I think this is a good idea, but causes unit tests to get unexpected log output.<br>Also it changes the application's user interaction behavior just by using a different libosmocore version, which we should avoid.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I think you need to solve this by adding a flag that keeps VTY command logging disabled by default, so that all the unit tests that work with both VTY and log output remain unchanged and keep passing.</p><p style="white-space: pre-wrap; word-wrap: break-word;">This could either be a boolean flag that the main() scope of each application sets to true in C,<br>and/or this could be set to true by an actual vty config item.</p><p style="white-space: pre-wrap; word-wrap: break-word;">An example for such a flag would be osmo_fsm_term_safely(), e.g. set to true in msc_main.c, but off by default to not cause inadvertent behavioral changes from a new libosmocore version.</p><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/14974">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/14974">change 14974</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/libosmocore/+/14974"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I5f6639f6ebc3e8f8df50ef48538df94285b759fd </div>
<div style="display:none"> Gerrit-Change-Number: 14974 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 30 Jul 2019 16:57:53 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>