<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1: Code-Review-1</p><p style="white-space: pre-wrap; word-wrap: break-word;">The removed options were marked as deprecated so I also don't think reverting the change is the right way. Unfortunately we don't have releases in osmocom-bb which would allow us to handle deprecations better.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">With Linux Kernel project in our genes let me appeal to the "no userspace regression" mentality... Something that was actively used got broken. Instead of trying to explain of how it was just "deprecated" or how gray the snow is and this three headed monkey behind you..., let's own the problem.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Don't be afraid to revert quickly and show some sympathy/empathy with the users of a system.</p><p style="white-space: pre-wrap; word-wrap: break-word;">And on the topic of deprecation:</p><p style="white-space: pre-wrap; word-wrap: break-word;">1st) Passing --vty_port=X will fail config parsing and lead to the process to exit. This was noticeable to third party applications.</p><p style="white-space: pre-wrap; word-wrap: break-word;">2nd) Deprecation gives a grace period o someone to move somewhere else, find a replacement. None was given...<br></p><p><a href="https://gerrit.osmocom.org/12208">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12208">change 12208</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/12208"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmocom-bb </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ie89bc16b667dbe05baa76cfa4f86b4946f8019e8 </div>
<div style="display:none"> Gerrit-Change-Number: 12208 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Holger Freyther <holger@freyther.de> </div>
<div style="display:none"> Gerrit-Reviewer: Holger Freyther <holger@freyther.de> </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: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-CC: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 18 Dec 2018 16:07:08 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>