<p><a href="https://gerrit.osmocom.org/c/osmo-ci/+/15761">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/osmo-ci/+/15761/2/docker/Dockerfile_osmocom_jenkins.amd64">File docker/Dockerfile_osmocom_jenkins.amd64:</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/osmo-ci/+/15761/2/docker/Dockerfile_osmocom_jenkins.amd64@14">Patch Set #2, Line 14:</a> <code style="font-family:monospace,monospace">            asciidoc \</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">damn what a pity […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">This is funny, according to the linked thread, it should *not* work since bash would parse it as one line, probably like this:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;"> asciidoc-dblatex #Required by osmocom-bb autoconf</pre><p style="white-space: pre-wrap; word-wrap: break-word;">However, when I tried it out, it did work. That must be, because the docker parser removes the comments before passing it to the shell.</p><p style="white-space: pre-wrap; word-wrap: break-word;">So I've thought about updating the patch. If we do it like that, we would need to remove the alphabetic order again, and we would basically have the categories of dependencies as we had before. But they don't seem to be so useful to me, we had "generic" dependencies at the top (without any comment), then later "still generic", "building", "and all" which are not meaningful. The others are more or less dependencies for one specific project, but still with either duplicated or an incomplete list of dependencies.</p><p style="white-space: pre-wrap; word-wrap: break-word;">With this in mind, my personal preference would be having this alphabetically sorted list without duplicates, and in the future being able to look at why dependencies were added with git blame.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ci/+/15761">change 15761</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/c/osmo-ci/+/15761"/><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-Change-Id: I190e62dd39533b284c93107607dac24ece6d7be6 </div>
<div style="display:none"> Gerrit-Change-Number: 15761 </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-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 11 Oct 2019 11:54:53 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Comment-In-Reply-To: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>