<p><a href="https://gerrit.osmocom.org/10862">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/10862/6/debian-repo-install-test/testdata/repo-install-test.sh">File debian-repo-install-test/testdata/repo-install-test.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/10862/6/debian-repo-install-test/testdata/repo-install-test.sh@49">Patch Set #6, Line 49:</a> <code style="font-family:monospace,monospace">   osmo-bsc --version</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">it's not really the task at hand, but in the past we also had the problem that some packages would generate UNKNOWN as version string.  So as we're calling all binaries with --version anyway, it might be a good idea to also check that there's no UNKNOWN string present in the output?</p><p style="white-space: pre-wrap; word-wrap: break-word;">I guess the best would be to add the above new test as a new function, so the test output (ends up in jenkins console log) would clearly state what failed.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Feel free to come up with whatever way you think is best, and we should do this in a follow-up patch to this current patch.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/10862">change 10862</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/10862"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: docker-playground </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I0f0a8b4f0ccd706cd1af98fe25accd6fd410e007 </div>
<div style="display:none"> Gerrit-Change-Number: 10862 </div>
<div style="display:none"> Gerrit-PatchSet: 6 </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, 13 Sep 2018 14:14:12 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>