<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 3:</p><p style="white-space: pre-wrap; word-wrap: break-word;">I guess this means that in any kind of cross compilation environment we don't get manuals anymore?  How do we handle this gracefully?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">osmo-bsc.git already has this patch applied, so if this needs a change, osmo-bsc will need the same change.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I assumed the primary source of the vty reference is the build job for http://ftp.osmocom.org/docs/latest/</p><p style="white-space: pre-wrap; word-wrap: break-word;">Another way to guarantee up-to-date-ness *and* presence of vty reference is to always keep a copy in the source tree. The gerrit build job could then regenerate and fail if there is a difference. That way a vty ref is always present.</p><p style="white-space: pre-wrap; word-wrap: break-word;">One aspect there is that limited builds (e.g. the --disable-iu build in osmo-msc) produce a reduced vty reference that doesn't match the full one saved in the source tree. The gerrit verification job would have to make sure to only check the vty reference in the maximally featured build. possible.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I thought it would be more convenient and less spent time overall to not have to update the vty reference in the source tree and just accept that we don't build a reference during cross compilation. If we're willing to spend more time (now and in the future) then we could keep a copy in the source, too, and reject mismatching vty reference in gerrit.</p><p><a href="https://gerrit.osmocom.org/c/osmo-mgw/+/18927">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-mgw/+/18927">change 18927</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-mgw/+/18927"/><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-Change-Id: I526af21134087e2b43b9ada59c93f636ae242e24 </div>
<div style="display:none"> Gerrit-Change-Number: 18927 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 23 Jun 2020 12:57:36 +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>