<p><a href="https://gerrit.osmocom.org/12694">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/12694/1//COMMIT_MSG">Commit Message:</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/12694/1//COMMIT_MSG@9">Patch Set #1, Line 9:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Remove hardcoded /usr/local/bin/osmo-* run commands from jenkins.sh.<br>This caused the *-latest tests to fail, as they have the Osmocom<br>binaries in /usr/bin instead of /usr/local/bin.<br><br>Adjust Dockerfiles where the default run command was different from the<br>hardcoded run command in jenkins.sh (e.g. missing config file<br>parameters).<br></pre></blockquote></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">how are those two seemingly unrelated topics connected to each other? Shouldn't this be two commits?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">They are connected, because after removing the hardcoded command from jenkins.sh, it will fall back to the default command in Dockerfile. But you are right, these are two logical changes, made a second commit.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/12694/1//COMMIT_MSG@17">Patch Set #1, Line 17:</a> <code style="font-family:monospace,monospace">Related: OS#367</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">probably missing a digit, it's long ago since we had three-digit issue numbers...</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Oh right, fixed!</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12694">change 12694</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/12694"/><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: I9d1427e66028ffd15fed211b2dab5adc37718666 </div>
<div style="display:none"> Gerrit-Change-Number: 12694 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 25 Jan 2019 13:23:00 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>