<p><a href="https://gerrit.osmocom.org/12571">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/12571/1//COMMIT_MSG">Commit Message:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/12571/1//COMMIT_MSG@9">Patch Set #1, Line 9:</a> <code style="font-family:monospace,monospace">This simplifies tests refactoring by showing exact byte where mismatch</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">> that's technically not allowed […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Max, I'm really not happy about you repeatedly trying to change how things are done.  In osmocom, we never have had any multiline logging.  This ispart of the log file format. Whether or not it's not technically posibble, you should have concluded from the fact that it's not done anywhere that it's not desired/intended. I'm actually going to open a bug report about this now.</p><p style="white-space: pre-wrap; word-wrap: break-word;">How do you expect any automatic log post-processing to work when suddenly there are log lines that don't follow the usual formatting?</p><p style="white-space: pre-wrap; word-wrap: break-word;">Have you validated/verified that e.g. the  GSMTAP based logging still works with this multi-line approach?  Have you verified that the rendering e.g. in the GSMTAP log dissector for wireshark works? And there are more questions like this - whihc I all find rather rhethorically because it simply shouldn't be done in the first place.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I believe the PCU did that at some point in the past or maybe still using multi-line log strings.  If so, they should equally be resolved/reoved.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12571">change 12571</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/12571"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-msc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: If9771c973f2bc55580f4c146bdbeeb1609d56786 </div>
<div style="display:none"> Gerrit-Change-Number: 12571 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Max <msuraev@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Max <msuraev@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Stefan Sperling <stsp@stsp.name> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 29 Jan 2019 12:48:59 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>