<p><a href="https://gerrit.osmocom.org/10932">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/10932/2/scripts/osmo-depcheck/buildstack.py">File scripts/osmo-depcheck/buildstack.py:</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/10932/2/scripts/osmo-depcheck/buildstack.py@97">Patch Set #2, Line 97:</a> <code style="font-family:monospace,monospace">        old = os.environ[env_var] if env_var in os.environ else ""</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Nice trick, but that seems to be a bit over-engineered here IMHO. ":path" will work just as well.</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">hmm, really, ":/path" is allowed? but it looks unhappy. I don't think it's overengineered, but whatever, nm</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/10932/2/scripts/osmo-depcheck/config.py">File scripts/osmo-depcheck/config.py:</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/10932/2/scripts/osmo-depcheck/config.py@18">Patch Set #2, Line 18:</a> <code style="font-family:monospace,monospace"># they are mentioned with PKG_CHECK_MODULES in configure.ac.</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Good point with the tuples. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">not important.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/10932/2/scripts/osmo-depcheck/dependencies.py">File scripts/osmo-depcheck/dependencies.py:</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/10932/2/scripts/osmo-depcheck/dependencies.py@20">Patch Set #2, Line 20:</a> <code style="font-family:monospace,monospace">    # Clone when needed</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Well, this is a pretty short function (14 LOC), and it does have a few key differences: […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">not that important, keep it</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/10932">change 10932</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/10932"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ci </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I8f495dbe030775f66ac125e60ded95c5d7660b65 </div>
<div style="display:none"> Gerrit-Change-Number: 10932 </div>
<div style="display:none"> Gerrit-PatchSet: 5 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 20 Sep 2018 15:20:15 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>