<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">will this commit alone make it work?  If yes, we can merge the<br>patch as-is.  If not (which I suspect), then this patch should<br>either be the very last patch in the series, or it should at<br>least<br>for now still have an "#if 1" -> print error message like now.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Well it makes sense from chronological and development point of<br>view to have this first, since you want to build the new code using<br>the pre-existing logic of enabling it from a VTY cmd, so I first<br>need to enable it to test it. Once the whole set of patches in this<br>patchset is merged (they can be merged together), then Osmux works<br>again, at least for the most common cases I tested.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Since it's quite a lot of work, my plan is to have this patchset<br>merged providing initial support and once merged provide TTCN3<br>tests on osmo-bsc/osmo-msc/osmo-mgw. Later on features/special<br>cases can be improved, otherwise the amount of patches I need to<br>maintain in my branch is quite big.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">In the end, our software is difficult enough to use as-is.  We shouldn't confuse the user with options that can be enabled and imply something is working, while it absolutely isn't.  I'm sure we have already some of these, but we shouldn't add more to it.</p><p style="white-space: pre-wrap; word-wrap: break-word;">If you first enable some VTY option, commit that and then only later actually fix/implement it, then there's also the danger that we might tag a release in between.  The VTY commands then get listed in the manuals, etc.</p><p style="white-space: pre-wrap; word-wrap: break-word;">So in summary, I think it's not too much to ask to keep the vty/config options in a private branch until the functionality is actually known wokring.</p><p><a href="https://gerrit.osmocom.org/14021">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/14021">change 14021</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/14021"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-mgw </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ica2f82473bf1934502444be2325ee2049d938781 </div>
<div style="display:none"> Gerrit-Change-Number: 14021 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 14 May 2019 10:03:34 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>