<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:</p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm missing some kind of rationale. Is this what other osmo-* projects do?  Why is docker needed here? ...</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Yeah, every other project uses it:<br>$ find . -name regen_doc.sh -exec grep -l DOCKER_PLAYGROUND {} \;<br>./osmo-ggsn/doc/manuals/regen_doc.sh<br>./osmo-bts/doc/manuals/regen_doc.sh<br>./osmo-pcu/doc/manuals/regen_doc.sh<br>./osmo-mgw/doc/manuals/regen_doc.sh<br>./osmo-msc/doc/manuals/regen_doc.sh<br>./osmo-bsc/doc/manuals/regen_doc.sh<br>./osmo-cbc/doc/manuals/regen_doc.sh<br>./osmo-sgsn/doc/manuals/regen_doc.sh<br>./osmo-gbproxy/doc/manuals/regen_doc.sh<br>./osmo-hlr/doc/manuals/regen_doc.sh</p><p style="white-space: pre-wrap; word-wrap: break-word;">The issue the current script has is that it uses whatever osmo-sgsn is in you path, so the version will most likely mismatch (and it will fail if the version found in path can not read the config file from examples).</p><p style="white-space: pre-wrap; word-wrap: break-word;">What I can't explain is why simply calling ../../src/osmo-sgsn in regen_doc.sh won't work as well as using docker.</p><p><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/23305">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/23305">change 23305</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/+/23305"/><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: I09b01d2e4bc7b1c17588100b16ac9f04cf72cf99 </div>
<div style="display:none"> Gerrit-Change-Number: 23305 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: daniel <dwillmann@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-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 10 Mar 2021 13:12:10 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>