<p style="white-space: pre-wrap; word-wrap: break-word;">Looks okay comparing it to an existing jenkins.sh in osmo-bsc</p><p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4;">Code-Review +1</span></p><p><a href="https://gerrit.osmocom.org/11549">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/11549/1/contrib/jenkins.sh">File contrib/jenkins.sh:</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/11549/1/contrib/jenkins.sh@42">Patch Set #1, Line 42:</a> <code style="font-family:monospace,monospace">        $MAKE distcheck || cat-testlogs.sh</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">cat-testlogs.sh wouldn't actually do anything useful since we don't have tests (yet?), but it shouldn't do any harm, either. If you want to leave it in for future tests why not leave the make check call in as well since there is already an empty testsuite?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11549">change 11549</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/11549"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-pcap </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I955b99ce27df143f5d022619dd14e32b763e6c14 </div>
<div style="display:none"> Gerrit-Change-Number: 11549 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 31 Oct 2018 18:16:28 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>