<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;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Patch Set 3:<br>> > I don't see a beneficial use case in clearing the IPA name<br>before<br> > saving the current config. Why would anyone want to omit that<br>name?<br> > If we require the name to be stored in the config file, why<br>should<br> > it not always be saved when the file gets written out?<br>><br>I don't get you there. I don't see anywhere that it is REQUIRED<br>that the name is stored in the config file. That's why we have<br>defaults right? When storing the config back into the cfg file we<br>usually have same approach. If it's the default value, don't save<br>it, because it means the user simply wants the default value and we<br>may decide later that another default value is better, this way the<br>user can benefit from it for free. It also keeps config files<br>shorter and easy to find "user specific values".</pre></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">But... wait... my patch saves the ipa-name to the config file only<br>if the user has overridden it. If the user does not change it, the<br>'ipa-name' line won't appear in the saved configuration.</p><p style="white-space: pre-wrap; word-wrap: break-word;">You sound like you believed we'd save the default name<br>"MSC-00-00-00-00-00-00"?<br>That is not the case!</p><p style="white-space: pre-wrap; word-wrap: break-word;">Does this address your concern or did I misunderstand you?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I was just replying that I still lack a "no ipa-name" in order to be able to go back to default before saving the config, but I didn't really understand your arguments against it, I was just stating I don't get them and trying to explain why. I know your patch currently doesn't do that.</p><p><a href="https://gerrit.osmocom.org/12243">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12243">change 12243</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/12243"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-sgsn </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ib2f65fed9f56b9718e8a9647e3f01dce69870c1f </div>
<div style="display:none"> Gerrit-Change-Number: 12243 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Stefan Sperling <stsp@stsp.name> </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: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Stefan Sperling <stsp@stsp.name> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 17 Dec 2018 15:33:35 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>