<p><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/15887">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/osmo-sgsn/+/15887/1/doc/examples/osmo-gbproxy/osmo-gbproxy.cfg">File doc/examples/osmo-gbproxy/osmo-gbproxy.cfg:</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/osmo-sgsn/+/15887/1/doc/examples/osmo-gbproxy/osmo-gbproxy.cfg@a23">Patch Set #1, Line 23:</a> <code style="font-family:monospace,monospace"> encapsulation framerelay-gre enabled 0</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">It seems like it, but it will fail unless both lines are removed.</blockquote></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">But then it fails somehow later? I don't understand, because the first removed cmd cannot fail with a warning:<br>"""<br>DEFUN(cfg_frgre_enable, cfg_frgre_enable_cmd,<br>      "encapsulation framerelay-gre enabled (1|0)",<br> ENCAPS_STR "NS over Frame Relay over GRE Encapsulation\n"<br>   "Enable or disable Frame Relay over GRE\n"<br>  "Enable\n" "Disable\n")<br>{<br>        int enabled = atoi(argv[0]);</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">  vty_nsi->frgre.enabled = enabled;</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">  return CMD_SUCCESS;<br>}<br>"""</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><br>While the second indeed does print the warning you see:<br>"""<br>DEFUN(cfg_frgre_local_ip, cfg_frgre_local_ip_cmd,<br>      "encapsulation framerelay-gre local-ip A.B.C.D",<br>        ENCAPS_STR "NS over Frame Relay over GRE Encapsulation\n"<br>   "Set the IP address on which we listen for NS/FR/GRE\n"<br>     "IP Address\n")<br>{<br>  struct in_addr ia;</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">    if (!vty_nsi->frgre.enabled) {<br>             vty_out(vty, "FR/GRE is not enabled%s", VTY_NEWLINE);<br>               return CMD_WARNING;<br>   }<br>     inet_aton(argv[0], &ia);<br>  vty_nsi->frgre.local_ip = osmo_ntohl(ia.s_addr);</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">   return CMD_SUCCESS;<br>}<br>"""</pre></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/15887">change 15887</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/osmo-sgsn/+/15887"/><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-Change-Id: Ibbb2fb80540ba78cded99ced1926e523d529dc35 </div>
<div style="display:none"> Gerrit-Change-Number: 15887 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 29 Oct 2019 11:15:27 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Comment-In-Reply-To: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>