<p><a href="https://gerrit.osmocom.org/12045">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/12045/2/jobs/gerrit-verifications.yml">File jobs/gerrit-verifications.yml:</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/12045/2/jobs/gerrit-verifications.yml@97">Patch Set #2, Line 97:</a> <code style="font-family:monospace,monospace">            (FIRMWARE_VERSION == "femtobts_v2.7" && BTS_MODEL == "sysmo" && with_manuals == True) ||</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">why building thm here instead of "master" version?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/12045/2/jobs/gerrit-verifications.yml@177">Patch Set #2, Line 177:</a> <code style="font-family:monospace,monospace">            (with_vty == "False" && with_dsp == "none" && FIRMWARE_VERSION=="master" && with_manuals == False) ||</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I'd also enable it here instead of the sysmo one.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/12045/2/jobs/gerrit-verifications.yml@190">Patch Set #2, Line 190:</a> <code style="font-family:monospace,monospace">            (IU == "--enable-iu" && with_manuals == True) ||</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Since enable iu takes more time, probably makes sense to enable document build in disable-iu instead, so they take more a less the same time when running in parallel.</p><p style="white-space: pre-wrap; word-wrap: break-word;">You can probably do the same for other projects, like osmo-ggsn (enable-gtp), etc.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12045">change 12045</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/12045"/><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: Ie0643b17a6be6e86e281b50d78dd7a6858cf99ee </div>
<div style="display:none"> Gerrit-Change-Number: 12045 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 30 Nov 2018 13:08:07 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>