<p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/18374">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-bsc/+/18374/2/doc/manuals/Makefile.am">File doc/manuals/Makefile.am:</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-bsc/+/18374/2/doc/manuals/Makefile.am@24">Patch Set #2, Line 24:</a> <code style="font-family:monospace,monospace">$(top_builddir)/src/osmo-bsc/osmo-bsc</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Then afaiu Make should already handle dependencies correctly and there's no need for this patch?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Am not sure I get what you're saying, pespin.<br>The point is to build the bsc_vty_reference.xml, for that we need osmo-bsc. How does that not need this patch?</p><p style="white-space: pre-wrap; word-wrap: break-word;">Make's multi-dir support isn't great. Depending on osmo-bsc here is more about re-generating the XML file when the binary has changed (and not every time). We would indeed cause a 'make -C src/osmo-bsc osmo-bsc' but I fear that that fires a separate make process. BTW, this is not a situation specific to this patch, this same thing happens for all of our binaries (e.g. unit tests) requiring lib files from elsewhere in the source tree. So far it seems to work out ok in our builds by naming the subdir that builds the dependencies first. I think this is one reason why so many build systems other than 'make' were written.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/18374">change 18374</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-bsc/+/18374"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I5dc872149154e1a949bb6a2b9bbc1461e0fc51f6 </div>
<div style="display:none"> Gerrit-Change-Number: 18374 </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: fixeria <axilirator@gmail.com> </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 20:34:00 +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: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>