<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 2:</p><p style="white-space: pre-wrap; word-wrap: break-word;">That's what I'm not getting. We either move to generate it from builddir or keep it generated in srcdir, but why checking both? I'm simply asking if you find any reason for that.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Obviously, if we change osmo-gsm-manuals.git now to only build the XML ref from binary, we would be forced to change every osmo*.git at the same time. We are not ready to build *every* osmo program's XML ref from built binary:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">- We first need to introduce --vty-ref-xml cmdline switches in every program.<br>- Switching to generate XML during build also requires individual makefile rules<br>  in every git repos -- obviously osmo-gsm-manuals cannot know the name and location<br>  of all osmo binaries everywhere.</pre><p style="white-space: pre-wrap; word-wrap: break-word;">I believe the best way forward is to still find the XML reference in the source tree by default.<br>Each git repos that is ready can switch to generating from binary during build time when it is ready.<br>Then we don't need to do one massive switch that is then backwards incompatible.</p><p style="white-space: pre-wrap; word-wrap: break-word;">(Even if we would, in a massive effort, switch all our git repositories to generating XML ref at build time, then we would not be able to build older manuals with osmo-gsm-manuals master. Plus, any repos out there that we forgot about, or that someone else may have created that is depending on osmo-gsm-manuals would stop working all of a sudden.)</p><p style="white-space: pre-wrap; word-wrap: break-word;">Hence this patch introduces a BUILT_REFERENCE_XML to add a dependency in case the caller needs it. And also it looks in both srcdir and builddir for the *reference.xml file that would so far always be in srcdir.</p><p style="white-space: pre-wrap; word-wrap: break-word;">At first I had this solved by a simple</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">  $(ls -1 $(srcdir)/vty/*reference.xml $(builddir)/vty/*reference.xml)</pre><p style="white-space: pre-wrap; word-wrap: break-word;">but the added script makes sure that we don't barf confusing error output, and that we don't get confused by ls aliases or env config.</p><p><a href="https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/18362">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/18362">change 18362</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-gsm-manuals/+/18362"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-gsm-manuals </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I613d692328050a036d05b49a436ab495fc2087ba </div>
<div style="display:none"> Gerrit-Change-Number: 18362 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </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: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 25 May 2020 21:01:53 +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>